Closed
Bug 423351
Opened 17 years ago
Closed 17 years ago
changes to dateAdded and lastModified properties of bookmarks are sometimes not propagated to the UI
Categories
(Firefox :: Bookmarks & History, defect)
Firefox
Bookmarks & History
Tracking
()
RESOLVED
FIXED
People
(Reporter: dietrich, Unassigned)
References
Details
from https://bugzilla.mozilla.org/show_bug.cgi?id=393498#c3:
the UI will not be updated if only undo-ing a
cut of a few items. since the transaction count is less than
MIN_TRANSACTIONS_FOR_BATCH in nsPlacesTransactionsService, these changes are
wrapped up in a single transaction when cut/undo-ing of a single bookmark for
example. therefore there's no batching, and the front-end receives and
processes OnItemAdded before the date changes occur. we don't send
OnItemChanged for added/modified because mostly those values are changed as a
result of other changes that will send a specific notification.
the workaround is to cause the UI to reload the container (by clicking another
folder, and then returning, for example). you'll then see the restored dates.
one option might be to add aDoNotify to SetDateAdded/SetLastModified, however,
this is pretty clunky and serves only this unique case. another might be to
just get the date values directly from the bookmark service in treeView.js, but
given the quantity of UI updating in the treeview (every mouse move over the
cell, etc), that's likely going to cause performance issues.
Reporter | ||
Comment 1•17 years ago
|
||
fixed in bug 393498
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Comment 2•15 years ago
|
||
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h".
In Thunderbird 3.0b, you do that as follows:
Tools | Message Filters
Make sure the correct account is selected. Click "New"
Conditions: Body contains places-to-b-and-h
Change the action to "Delete Message".
Select "Manually Run" from the dropdown at the top.
Click OK.
Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter.
Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in
before you can comment on or make changes to this bug.
Description
•