Closed
Bug 1425290
Opened 7 years ago
Closed 4 years ago
Standardize default target folder also for "Bookmark This Link" and Bookmark this frame
Categories
(Firefox :: Bookmarks & History, defect, P3)
Tracking
()
VERIFIED
FIXED
84 Branch
| Tracking | Status | |
|---|---|---|
| firefox84 | --- | verified |
People
(Reporter: Virtual, Assigned: Gijs)
References
(Blocks 1 open bug)
Details
(7 keywords, Whiteboard: [fxsearch])
Attachments
(2 files)
Follow up of bug #1120110 to standardize default output folder for adding bookmarks by various methods, even by using "Bookmark This Link" item in hyperlink menu
Flags: needinfo?(mak77)
|
||
Comment 1•7 years ago
|
||
Forwarding to bbell, should be also unify the Bookmark This Link destination, or should we leave it alone?
Flags: needinfo?(mak77) → needinfo?(bbell)
|
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
|
Updated•7 years ago
|
status-firefox60:
--- → affected
|
||
Updated•7 years ago
|
Severity: major → normal
Priority: -- → P3
Whiteboard: [fxsearch]
|
|
||
Comment 3•7 years ago
|
||
All the bookmarking operations are currently going to unfiled, apart from Bookmark This Link and Bookmarks This Frame that are going to the menu.
I think we should unify them to unfiled bookmarks, then we can look into bug 1412263 to provide an option to change the default globally. Whether that option should be exposed an a Preference or hidden in about:config is still TBD.
Blocks: 1412263
|
|
||
Updated•7 years ago
|
Summary: Standardize default output folder by using "Bookmark This Link" item in hyperlink menu → Standardize default target folder also for "Bookmark This Link" and Bookmark this frame
|
|
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
|
Updated•7 years ago
|
status-firefox61:
--- → affected
|
|
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
|
Updated•6 years ago
|
status-firefox63:
--- → affected
|
||
Updated•6 years ago
|
|
|
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
|
Updated•6 years ago
|
status-firefox-esr60:
--- → affected
|
||
Updated•6 years ago
|
status-firefox64:
affected → ---
status-firefox65:
affected → ---
status-firefox-esr60:
affected → ---
|
|
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
|
Updated•6 years ago
|
status-firefox64:
--- → fix-optional
status-firefox65:
--- → fix-optional
status-firefox66:
--- → fix-optional
Since this bug is not a high priority, a regression, or a security issue, it's not necessary to track its status.
status-firefox58:
wontfix → ---
status-firefox59:
wontfix → ---
status-firefox60:
wontfix → ---
status-firefox61:
wontfix → ---
status-firefox62:
wontfix → ---
status-firefox63:
wontfix → ---
status-firefox64:
wontfix → ---
status-firefox65:
fix-optional → ---
status-firefox66:
fix-optional → ---
status-firefox67:
fix-optional → ---
QA Contact: Virtual
|
|
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
|
Updated•6 years ago
|
QA Contact: Virtual
|
|
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
|
Comment 5•5 years ago
|
||
Looks like Bryan Bell [:bbell] Bugzilla account is disabled, so I'm dropping needinfo? request.
@ Marco Bonardo [:mak] - As triage owner, do you know who could take this over? Thank you very much in advance!
Flags: needinfo?(bbell) → needinfo?(mak)
|
|
||
Comment 6•5 years ago
|
||
We don't have a UX referent atm, looking around for a future rresourced team to take care of these things.
Flags: needinfo?(mak)
Keywords: blocked-ux
|
Assignee | |
Comment 7•4 years ago
|
||
Depends on D94500
|
||
Updated•4 years ago
|
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 8•4 years ago
|
||
Depends on D94879
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/e6b38d403cc3
use the new default bookmark location pref for 'bookmark this link' and 'bookmark this frame', r=mak
https://hg.mozilla.org/integration/autoland/rev/cf70da964b70
also default bookmark properties (for keyword search bookmarks and multiple tab bookmarks) to the new default parent, r=mak
|
||
Comment 10•4 years ago
|
||
Backed out for bc failures on browser_bookmarkProperties_addKeywordForThisSearch.js
Backout link: https://hg.mozilla.org/integration/autoland/rev/d2c0ca2f17bd606bc01e65c6b62663da16a82488
Log link: https://treeherder.mozilla.org/logviewer?job_id=320868524&repo=autoland&lineNumber=4573
Flags: needinfo?(gijskruitbosch+bugs)
|
|
Assignee | |
Updated•4 years ago
|
Flags: needinfo?(gijskruitbosch+bugs)
|
||
Comment 11•4 years ago
|
||
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/ca3a56aafb99
use the new default bookmark location pref for 'bookmark this link' and 'bookmark this frame', r=mak
https://hg.mozilla.org/integration/autoland/rev/6354c3742a1a
also default bookmark properties (for keyword search bookmarks and multiple tab bookmarks) to the new default parent, r=mak
|
||
Comment 12•4 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/ca3a56aafb99
https://hg.mozilla.org/mozilla-central/rev/6354c3742a1a
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox84:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 84 Branch
|
|
Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)
Reporter
|
|
Comment 13•4 years ago
|
||
I'm confirming that bug is fixed, starting in Mozilla Firefox Nightly 84.0a1 (2020-11-06), so I'm marking this bug as VERIFIED.
Thank you very much! \o/
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•