Closed Bug 1966284 Opened 11 months ago Closed 7 months ago

hide new profile manager pages (new, edit, delete) from about:about

Categories

(Toolkit :: Startup and Profile System, defect)

defect

Tracking

()

RESOLVED FIXED
144 Branch
Tracking Status
firefox144 --- fixed

People

(Reporter: dveditz, Assigned: mlucks)

References

Details

(Whiteboard: [fidefe-profile-management])

Attachments

(1 file)

The about:about page lists the profile management pages about:deleteprofile, about:editprofile, about:newprofile and it shouldn't -- I think we want people to get to those pages from the about:profilemanager entry point (especially "new" which is supposed to involve launching a whole new copy of Firefox). Certainly they shouldn't be shown in Release or Beta builds where the new profilemanager is disabled and the links do nothing but add to the clutter.

Ironically, about:profilemanager itself is hidden from about:about. That was done in bug 1884471 because at the time it was unfinished and because autocomplete preferred it over the about:profiles people were intending. In builds where it does work (e.g. Nightly) we do probably want this to be shown in about:about and to autocomplete ahead of the old confusing about:profiles (but we should still hide it in builds where it's disabled if that's possible). Better to have one about:about entrypoint than three.

The fix is easy: add nsIAboutModule::HIDE_FROM_ABOUTABOUT to their about-redirector entries
https://searchfox.org/mozilla-central/rev/25a38ac3b851b63e7d85a850940c354609e8126d/browser/components/about/AboutRedirector.cpp#164-178

See Also: → 1966286

The severity field is not set for this bug.
:mossop, could you have a look please?

For more information, please visit BugBot documentation.

Flags: needinfo?(dtownsend)
Flags: needinfo?(dtownsend)
Whiteboard: [fidefe-profile-management]

The severity field is not set for this bug.
:mossop, could you have a look please?

For more information, please visit BugBot documentation.

Flags: needinfo?(dtownsend)
Severity: -- → S2
Flags: needinfo?(dtownsend)
Assignee: nobody → mlucks

Can this be landed this week so it doesn't persist into Firefox 144?

Pushed by kpatenio@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/b503d9fba942 https://hg.mozilla.org/integration/autoland/rev/214a32f546e2 Hide new profile manager pages from aboutabout - r=dveditz,profiles-reviewers,mossop,jhirsch
Status: NEW → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → 144 Branch
QA Whiteboard: [qa-triage-done-c145/b144]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: