Consider adding about:tor and others to the list of pages that do not need letterboxing

added component::applications/tor browser in Legacy / Trac owner::tbb-team in Legacy / Trac priority::medium in Legacy / Trac severity::normal in Legacy / Trac status::new in Legacy / Trac type::defect in Legacy / Trac labels

I quite like it being LB'ed as it shows something is off with the rounding on new windows (all those quirks with toolbar showing, DPI not at 100%, etc) - i.e. before you load web content. Note that new tabs can grab the non-LB'ed size before it renders (see TZP or use this -> https://thorin-oakenpants.github.io/testing/newwinLB.html )

moved from legacy/trac#32411 (moved)

added Bug label and removed 1 deleted label

removed 1 deleted label

added Icebox UX labels

changed title from Consider adding about:tor to the list of pages that do not need letterboxing to Consider adding about:tor and others to the list of pages that do not need letterboxing

marked this issue as related to #31064 (closed)

I think we should add also other pages, such about:torconnect and in general other pages that are considered privileged (#31064 (closed)).

It would be definitely a plus for the language notification box that we are adding for #41378 (closed), since it triggers letterbox.

assigned to @pierov

removed Icebox label

added Doing label

@pierov yes, this looks like a dupe of the more general (and old) #31064 (closed). Close?

Uhm. About pages can be set either for privileged or unprivileged, but the latter should be chosen, if possible.

However, I think it'd be safe to disable letterboxing for all about: pages, either privileged or unprivileged.

Also, not sure you already know it, but for these case GitLab has /dupe, that marks the issue also as a duplicate. It took me months to find it .

GitLab has /dupe, that marks the issue also as a duplicate. It took me months to find it .

Thanks for the hint, can't resist trying right now!

Ahahaha . Please notice that richard's patch above works, but I had some concerns about possible side effects.

There's a get needsLetterbox() { in browser/base/content/tabbrowser-tab.js, but the trivial way to modify it doesn't seem to work in my first test (I haven't cleaned the cache, though).