Closed
Bug 1463738
Opened 6 years ago
Closed 6 years ago
Remove the Bookmark description UI
Categories
(Firefox :: Bookmarks & History, enhancement, P1)
Firefox
Bookmarks & History
Tracking
()
RESOLVED
FIXED
Firefox 62
People
(Reporter: standard8, Assigned: standard8)
References
(Depends on 1 open bug)
Details
(Whiteboard: [fxsearch])
Attachments
(1 file)
Per bug 1402890, we are removing the description UI for bookmarks. This bug is for removing just the front-end parts, bug 1402890 will deal with the back-end cleanup in a release later than the UI is removed.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 2•6 years ago
|
||
Couple of notes: - browser/base/content/test/general/browser_bookmark_titles.js has tests that already cover the changes I did in PlacesCommandHook.bookmarkPage in browser-places.js. - The strings, annotations, sorting options I'll remove when we do bug 1402890.
Comment 3•6 years ago
|
||
mozreview-review |
Comment on attachment 8980596 [details] Bug 1463738 - Remove the Bookmark description UI. https://reviewboard.mozilla.org/r/246758/#review252986 LGTM except for the strings. I'm not too familiar with this code yet, so it's quite possible there's more stuff to remove that I'm not aware of. But if the UI seems to work and tests pass, I guess that's good enough for now, and we can always remove more leftovers later. ::: browser/components/places/content/editBookmarkPanel.inc.xul (Diff revision 1) > - > - <vbox id="editBMPanel_descriptionRow" > - collapsed="true"> > - <observes element="additionalInfoBroadcaster" attribute="hidden"/> > - <label value="&editBookmarkOverlay.description.label;" > - accesskey="&editBookmarkOverlay.description.accesskey;" Remove these strings and any others that become unused with this patch?
Attachment #8980596 -
Flags: review?(dao+bmo)
Comment 4•6 years ago
|
||
(In reply to Mark Banner (:standard8) from comment #2) > - The strings, annotations, sorting options I'll remove when we do bug > 1402890. Missed this. What's the rationale for keeping the strings?
Updated•6 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 5•6 years ago
|
||
mozreview-review-reply |
Comment on attachment 8980596 [details] Bug 1463738 - Remove the Bookmark description UI. https://reviewboard.mozilla.org/r/246758/#review252986 > Remove these strings and any others that become unused with this patch? Yeah, it was a just in case, but I'd forgotten that L10n has better ways of dealing with that these days. So I'll remove them.
Comment hidden (mozreview-request) |
Comment 7•6 years ago
|
||
mozreview-review |
Comment on attachment 8980596 [details] Bug 1463738 - Remove the Bookmark description UI. https://reviewboard.mozilla.org/r/246758/#review253680
Attachment #8980596 -
Flags: review?(dao+bmo) → review+
Pushed by mbanner@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e9a9f3abd889 Remove the Bookmark description UI. r=dao
Assignee | ||
Comment 9•6 years ago
|
||
Proposed relnote: The description field for bookmarks has been removed. Users that manually changed the field can export old descriptions via html [1] or json [2]. Descriptions will be removed in the next release. [1] https://support.mozilla.org/kb/export-firefox-bookmarks-to-backup-or-transfer [2] https://support.mozilla.org/kb/restore-bookmarks-from-backup-or-move-them#w_backup-and-restore
relnote-firefox:
--- → ?
Comment 10•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/e9a9f3abd889
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 62
Comment 11•6 years ago
|
||
(In reply to Mark Banner (:standard8) from comment #9) > Proposed relnote: > > The description field for bookmarks has been removed. Users that manually > changed the field can export old descriptions via html [1] or json [2]. > Descriptions will be removed in the next release. > > [1] > https://support.mozilla.org/kb/export-firefox-bookmarks-to-backup-or-transfer > [2] > https://support.mozilla.org/kb/restore-bookmarks-from-backup-or-move- > them#w_backup-and-restore Added to the release notes for Nightly 62 with this wording, I'll let the 62 release owner (CCed) decide on the inclusion and wording for beta and release
Updated•6 years ago
|
Assignee | ||
Comment 12•6 years ago
|
||
Pascal, please can you update the release notes to change "Descriptions will be removed in the next release" to "Stored descriptions will be removed in a future release". (xref bug 1402890 comment 30).
Flags: needinfo?(pascalc)
Comment 13•6 years ago
|
||
(In reply to Mark Banner (:standard8) from comment #12) > Pascal, please can you update the release notes to change "Descriptions will > be removed in the next release" to "Stored descriptions will be removed in a > future release". > > (xref bug 1402890 comment 30). Done
Flags: needinfo?(pascalc)
Comment hidden (metoo) |
You need to log in
before you can comment on or make changes to this bug.
Description
•