Closed Bug 1429364 Opened 2 years ago Closed 2 years ago
The "Save Link as" context menu option do not work in the "Firefox Notes" sidebar
[Affected versions]: - Firefox 57.0.4 and up [Affected Platforms]: - All Windows - All Mac - All Linux [Prerequisites]: - Have a Firefox profile with the latest version of the "Firefox Notes" add-on (2.2.0dev custom build from 01.10.2018) installed. - Have a website link copied to clipboard. [Steps to reproduce]: 1. Open the browser with the profiles from prerequisites. 2. Paste the link from prerequisites into the "Notes" sidebar. 3. Select the link and then right click on it. 4. Click the "Save Link as" context menu option and observe the behavior. [Expected result]: - The OS's specific dialog box is opened. [Actual result]: - Nothing happens. [Notes]: - Attached a screen recording of the issue.
This doesn't work for a normal HTML page opened in the sidebar either. It's not a WebExtensions specific issue, moving over to Firefox.
Component: WebExtensions: Untriaged → General
Product: Toolkit → Firefox
Gijs, do you know if we have a bug on file for this already?
(In reply to Panos Astithas [:past] (please ni?) from comment #2) > Gijs, do you know if we have a bug on file for this already? bug 1285012 is the closest, though it doesn't mention this particular item. The context menu in the sidebar has long been iffy for many things. I don't know who would be a good person to go through and audit it in all its forms. Bouncing this for you to be able to prio / assign someone / dupe / organize / etc. :-)
Flags: needinfo?(gijskruitbosch+bugs) → needinfo?(past)
Nice find! Based on Mike's work in that bug I have a pretty simple fix that seems to do the trick. Let's see what try thinks: https://treeherder.mozilla.org/#/jobs?repo=try&revision=2cc60d4c6dd067bfd1e7e9b8a46d573061966809
Assignee: nobody → past
Status: NEW → ASSIGNED
(In reply to Panos Astithas [:past] (please ni?) from comment #4) > Nice find! Based on Mike's work in that bug I have a pretty simple fix that > seems to do the trick. Let's see what try thinks: > > https://treeherder.mozilla.org/#/ > jobs?repo=try&revision=2cc60d4c6dd067bfd1e7e9b8a46d573061966809 Lovely. Pre-emptive r=me on that patch (also, nice all-green try!). Can we have a follow-up to add some tests for context-menu-in-sidebar stuff? I bet it's possible to adapt the current set of tests that we have for the content context menu so that they run the exact same tests against the sidebar as well, which might unearth more issues (though I'm not sure how many of the context menus we actually activate, I think we mostly check that their visibility/enabled state is updated correctly - that'd still be a win over the current state, though!) Also, from auditing based on that change, looks like view partial source is also broken: https://dxr.mozilla.org/mozilla-central/rev/c38d22170d6f27e94c3c53093215d20255fab27a/browser/base/content/nsContextMenu.js#884 I'll file a follow-up myself for that one.
Comment on attachment 8942252 [details] Bug 1429364 - Fix 'Save Link As' context menu option in sidebars. https://reviewboard.mozilla.org/r/212532/#review218246
Attachment #8942252 - Flags: review?(gijskruitbosch+bugs) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/392278bcc755 Fix 'Save Link As' context menu option in sidebars. r=Gijs
Woot! There's a lot more sidebar users these days thanks to WebExtensions.
(In reply to :Gijs from comment #5) > Can we have a follow-up to add some tests for context-menu-in-sidebar stuff? Filed bug 1430281.
This behavior is no longer reproducible, now the "Save Link as" context menu option works in the "Firefox Notes" sidebar Verified as fixed using latest Nightly 59.0a1 Build ID 20180114220708, on, Windows 10 x64, Ubuntu 14.04 x64 and Mac 10.12.
Seems late for 58 (not a new bug, and we're at RC week), so I'll call this wontfix.
You need to log in before you can comment on or make changes to this bug.