1483598 - Consider using “Remove from Address Bar” and “Add to Address Bar” for all Page Actions

Status: RESOLVED FIXED

(WebExtensions :: Frontend, enhancement, P3)

Description

[ExE Boss]

Right now, built‑in page actions use “Remove from Address Bar” and “Add to Address Bar”, whereas WebExtension page actions use “Don’t Show in Address Bar” and “Show in Address Bar” respectively.

Is there a reason why it’s different other than it just sort‑of happened and now there’s two separate strings for adding and removing page actions?

Comment 1

[Kris Maglione [:kmag]]

This seems like a bug, yes.

Comment 2

[Mike Conca [:mconca]]

Sending this to UX for a decision on the final copy.

Comment 3

[emanuela [ux]]

Let's match the built in strings.

Comment 4

[Drew Willcoxon :adw]

It's not a bug, it was a deliberate decision by UX when this originally landed.  Needinfo'ing Aaron and Bryan -- not saying we shouldn't do this, but they should be consulted at least

Comment 5

[Drew Willcoxon :adw]

IIRC the reasoning for the difference is that the built-in actions are (generally) always shown in the urlbar, but extension actions are more likely to be shown only in certain contexts.  So adding a built-in action adds it to the urlbar immediately, but adding an extension action only means that it's allowed to be shown in the urlbar when the extension so chooses.  If extension actions used "Add", it might be the case that nothing visibly happens after you click "Add".

Bug 1413574 has some context, but I think most of the discussion was offline.

Comment 6

[Aaron Benson]

Yeah, sorry I can't think of a good reason why they would be different. Lets match the strings as suggested.

Comment 7

[Mark Striemer [:mstriemer]]

Attachment: bug 1483598 - Match system and user extension add/remove page action strings r?mixedpuppy

Comment 8

[Shane Caraveo (:mixedpuppy)]

Comment on attachment 9006661 [details]
bug 1483598 - Match system and user extension add/remove page action strings r?mixedpuppy

Shane Caraveo (:mixedpuppy) has approved the revision.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3b5f59ae29ba
Match system and user extension add/remove page action strings r=mixedpuppy

Comment 10

[Drew Willcoxon :adw]

This wasn't the right way to fix this.  Now we have two menu items that say "remove" and two menu items that say "add", as you can see in the browser.xul changes this patch made.  One pair is for built-in actions, the other for extensions.  The appropriate pair is selected here by setting an attribute on the context menu popup: https://dxr.mozilla.org/mozilla-central/source/browser/base/content/browser-pageActions.js#825  And then one or the other pair is hidden in CSS.

Could you please file a bug to consolidate the pairs down to one and update that logic?

Comment 11

[Mark Striemer [:mstriemer]]

Ah, I saw there were different functions being called and figured we needed two. I filed bug 1488930 to clean that up.

Comment 12

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/3b5f59ae29ba