1601404 - Zoom "Reset" menu should be renamed "Actual Size"

Status: RESOLVED FIXED

(Firefox :: Menus, enhancement, P3)

Description

[Asif Youssuff]

Like Chromium and Safari.

The View > Zoom > Reset menu item should be renamed "Actual Size" in order to be more consistent with those browsers.

I kept this bug broad, but this can be refiled to Cocoa instead if it makes more sense for macOS, but I don't see a real reason why it shouldn't also apply to Windows and Linux.

Comment 1

[:Gijs (he/him)]

I'd want someone from UX to weigh in before doing anything here. Eric, are you the right person?

Comment 2

[Eric Pang [:epang] UX]

Spoke with Betsy since this is a string change. We're both good with the change to "Actual Size"

Comment 3

[:Gijs (he/him)]

Note that when bug 1588895 lands, this would say "Actual Size" when what it does is reset to global zoom, so if your global zoom preference is 150%, it'd set it to that - not 100%. That seems to be what other browsers do too, but it does feel odd to call it "Actual Size"...

Comment 4

[:Gijs (he/him)]

Anyway, to fix this, we'd need to:

1. change the identifier of the string at https://searchfox.org/mozilla-central/source/browser/locales/en-US/browser/menubar.ftl#123-125 - probably menu-view-full-zoom-actual-size would be a good choice
2. change the label and access key properties to have correct values
3. use menu-view-full-zoom-actual-size at https://searchfox.org/mozilla-central/rev/ea63a0888d406fae720cf24f4727d87569a8cab5/browser/base/content/browser-menubar.inc#182 .

Comment 5

[Tanner Davies]

Could I take this bug, please?

Comment 6

[Tanner Davies]

Attachment: Bug 1601404 - Change View>Zoom>Reset in menu bar to "Actual Size" to be consistent with other browsers r=Gijs

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9ab1ac74d4a4
Change View>Zoom>Reset in menu bar to "Actual Size" to be consistent with other browsers r=Gijs,fluent-reviewers

Comment 8

[:Gijs (he/him)]

Thanks for the patch!

Comment 9

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/9ab1ac74d4a4