"Bookmark pages" context menu item from Library - History doesn't work
Categories
(Firefox :: Bookmarks & History, defect, P2)
Tracking
()
People
(Reporter: zstimi, Assigned: standard8)
Details
Attachments
(2 files)
Affected versions
- Fx 75.0b3
- Fx 76.0a1
Affected platforms
- Windows 10 x64
- Windows 8.1 x64
- Ubuntu 18.04 x64
- macOS 10.13
Steps to reproduce
- Launch Firefox, use a dirty profile with reasonable amount of browsing history.
- Select the History option from the View History, saved bookmarks and more Menu.
- Click on the Show All History menu item.
- Select 2 or more websites pressing CTRL from keyboard.
- Right click on selected items to open context menu.
- Click on Bookmark Pages.
Expected result
- The selected websites bookmarked.
Actual result
- Nothing happened, the selected pages are not bookmarked.
Regression range
- I will come back with regression range ASAP.
Additional Notes
- The selected websites are bookmarked if you insert some tags.
Comment 1•5 years ago
|
||
So this basically means adding multiple bookmarks from an history view is broken. I don't see errors reported to the console.
Mark, do you have any spare time to investigate this?
Assignee | ||
Comment 2•5 years ago
|
||
I've taken a quick look, and I can see that this._view.selectedNode
is null when there's more than one selected node:
The strange thing is, this was added back in bug 422181 in Firefox 3.6... and that code hasn't changed since then.
Timea, do you have a regression range yet? I think that might help here.
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Reporter | ||
Comment 3•5 years ago
|
||
Hi, I tested this issue on builds from Fx 48.0 to Fx 75.0b6 and it seems this is not a regression, reproduced on every build. I will attache a screenshot.
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 4•5 years ago
|
||
Assignee | ||
Updated•5 years ago
|
Comment 6•5 years ago
|
||
bugherder |
Updated•5 years ago
|
Comment 7•5 years ago
|
||
The patch landed in nightly and beta is affected.
:standard8, is this bug important enough to require an uplift?
If not please set status_beta
to wontfix
.
For more information, please visit auto_nag documentation.
Reporter | ||
Comment 9•5 years ago
|
||
I can confirm this issue is fixed, I verified using Firefox 77.0a1 (latest nightly 2020.05.03) on Win 8.1 x64, macOS 10.15 and Ubuntu 18.04 x64.
Description
•