In addition to the DND feedback, this patch:
- introduces a new var 'BookmarksInsertion' that 'encapsulates' DND feedback
display.
- rewrite the rtl handling related to DND. BookmarksInsertion deals with this
subtleties and the "public" feedback orientation ("before"|"on"|"after")
returned by BookmarksInsertion.getOrientation refers now to the DOM.
- paste and copy commands on the menus now insert after the rclicked element.
The hack that appended everything at the end of the container has been removed
or more accurately replaced by another one that is at least invisible to the
user and we already used for DND. The problem is due a problem with the content
building of containers with siblings (open in tabs stuff) not generated by the
bookmarks template.
- usual small glitch fixing.
git-svn-id: svn://10.0.0.236/trunk@161271 18797224-902f-48f8-a5cc-f745e15eee43
192624 - empty bookmark folder should show "(Empty)"
202798 - customizing toolbar in fullscreen mode breaks menus (bandaid)
221482 - bookmarks toolbar temporarily cleared when clicking restore default set in customize
232089 - bookmarks 'create in' shouldn't persist
234089 - bookmarks toolbar folder shouldn't be deletable with C-x (crasher)
238661 - moving a bookmark folder to itself deletes the folder
250179 - bookmarks toolbar wonky after customization
250820 - toolbar icon state/context menu/etc. broken after returning from fullscreen mode
251988 - livemark menu doesn't work after going to page without livemark and clicking
251335 - bookmarks toolbar folder has different context menu from other bookmark folders
251802 - refresh livemark doesn't work from toolbar/menu
252053 - make bookmarks use SafeOutputStream (dwitte)
252133 - new bookmarks items should be added below selected items, not above
252792 - livemark context menu contains active new folder menu item
252870 - context menu of empty personal toolbar folder has inconsistent items
252875 - bookmarks root name not localizable
git-svn-id: svn://10.0.0.236/trunk@160122 18797224-902f-48f8-a5cc-f745e15eee43
I didn't noticed that because linux doesn't honor the option "resizable=no".
Patch by Steffen Wilberg, r=me.
git-svn-id: svn://10.0.0.236/trunk@150211 18797224-902f-48f8-a5cc-f745e15eee43
Featuring an expandable tree and a dropdown menu containing the last used folders.
The 'new folder' functionality has not been included, yet.
Thanks to bz for the help on nsCOMArrays.
git-svn-id: svn://10.0.0.236/trunk@150142 18797224-902f-48f8-a5cc-f745e15eee43
PrintUtils.printPreview(enterPPCallback, exitPPCallback) where the optional callback functions are called just before entering and after exiting PP mode
- disable all short keys in PP mode except ctrl-W and fix bug 178485 and bug 215277. Now clicking the close window icon actually closes the window.
- global vars gPrintSettingsAreGlobal, gSavePrintSettings, gPrintSettings, gOldCloseHandler, gInPrintPreviewMode, gWebProgress are removed.
- A lot of cleanup.
git-svn-id: svn://10.0.0.236/trunk@148474 18797224-902f-48f8-a5cc-f745e15eee43
- enhance dumpOBJ debug routine to deal with null and properties not accessible that trigger an error
- enclose all the debug routine by a #ifdef DEBUG
git-svn-id: svn://10.0.0.236/trunk@148271 18797224-902f-48f8-a5cc-f745e15eee43
Also remove the bookmark edit command (ex: cmd_bm_paste) and use the edit ones instead (ex: cmd_paste)
git-svn-id: svn://10.0.0.236/trunk@147945 18797224-902f-48f8-a5cc-f745e15eee43
- logic change: a transaction will not be performed when one of its item can not be acted on. Before, the only the valid items were processed. A LOT of simplifications have come up. I am slowly moving toward the resolution of bug 168411, by simplifying the interface that will be needed to move the transactions to the bookmarks service.
- fix bug 221550: the selection is not retained when performing an invalid transaction
- no need to restore the selection after an action that doesn't touch the datasource.
- during a batch, the bookmarks toolbar observer wasn't notified.
- much faster preUpdateTreeSelection
- various code cleanup
- DND bookmarks in tree is now significantly faster.
- still need to figure out what is the best BATCH_LIMIT value.
git-svn-id: svn://10.0.0.236/trunk@147752 18797224-902f-48f8-a5cc-f745e15eee43
an additional fix is included: if no ...#BookmarksToolbarFolder arc is found, then search for an (obsolete) resource "NC:PersonalToolbarFolder" (221410).
When none of them is found, then a new btf is created (only once!).
git-svn-id: svn://10.0.0.236/trunk@147747 18797224-902f-48f8-a5cc-f745e15eee43
- remove unused functions. Note that RDF commands were already broken, so deleteBookmarkItem and insertBookmarkItem (that were not used) may come back, but in another form.
- remove the "New Bookmark Folder" and "New Search Folder"
- refactor the "html" serializer to avoid some duplicated code.
- remove the use of the profile name for the bookmarks root.
- enforce synthax "aArgument" in some random places.
git-svn-id: svn://10.0.0.236/trunk@147534 18797224-902f-48f8-a5cc-f745e15eee43
- fixing a potential crash in nsBookmarksService::GetTarget (when the RDF resource is null)
- remove unused function isChildOfContainer and GetResource in bookmarks.js though we may need later the latter.
git-svn-id: svn://10.0.0.236/trunk@147460 18797224-902f-48f8-a5cc-f745e15eee43
- fix the use of the immutable bookmarks.
- remove unused methods: GetLastCharset and AddBookmarkImmediately
- porting several small fixes mainly from Jan's work
- more prototyping for BookmarkTransaction
git-svn-id: svn://10.0.0.236/trunk@147445 18797224-902f-48f8-a5cc-f745e15eee43
