Closed Bug 1622243 Opened 4 years ago Closed 4 years ago

"Bookmark pages" context menu item from Library - History doesn't work

Categories

(Firefox :: Bookmarks & History, defect, P2)

75 Branch
defect
Points:
3

Tracking

()

RESOLVED FIXED
Firefox 77
Iteration:
77.1 - Apr 6 - Apr 19
Tracking Status
firefox75 --- wontfix
firefox76 --- wontfix
firefox77 --- verified

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

  1. Launch Firefox, use a dirty profile with reasonable amount of browsing history.
  2. Select the History option from the View History, saved bookmarks and more Menu.
  3. Click on the Show All History menu item.
  4. Select 2 or more websites pressing CTRL from keyboard.
  5. Right click on selected items to open context menu.
  6. 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.

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?

Flags: needinfo?(standard8)

I've taken a quick look, and I can see that this._view.selectedNode is null when there's more than one selected node:

https://searchfox.org/mozilla-central/rev/3f8c67d7fd836d559491e3fe497bc739f707c1a6/browser/components/places/content/controller.js#207

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.

Flags: needinfo?(standard8) → needinfo?(timea.zsoldos)
Has Regression Range: --- → no
Has STR: --- → yes
Priority: -- → P2

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.

Flags: needinfo?(timea.zsoldos)
Assignee: nobody → standard8
Status: NEW → ASSIGNED
Iteration: --- → 76.2 - Mar 23 - Apr 5
Points: --- → 3
Iteration: 76.2 - Mar 23 - Apr 5 → 77.1 - Apr 6 - Apr 19
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/71de2075e13e
Implement the bookmark pages option in the library window. r=mak
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 77

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.

Flags: needinfo?(standard8)

This can ride the trains.

Flags: needinfo?(standard8)

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.

You need to log in before you can comment on or make changes to this bug.