Non-active Profiler toolbar icon and text are mirrored incorrectly when placed in overflow menu
Categories
(DevTools :: Performance Tools (Profiler/Timeline), defect)
Tracking
(firefox-esr68 unaffected, firefox77 unaffected, firefox78 verified, firefox79 verified)
Tracking | Status | |
---|---|---|
firefox-esr68 | --- | unaffected |
firefox77 | --- | unaffected |
firefox78 | --- | verified |
firefox79 | --- | verified |
People
(Reporter: Fanolian+BMO, Assigned: gregtatum)
References
(Regression)
Details
(Keywords: nightly-community, regression, reproducible)
Attachments
(2 files)
7.27 KB,
image/png
|
Details | |
47 bytes,
text/x-phabricator-request
|
jcristau
:
approval-mozilla-beta+
|
Details | Review |
Thank you for helping make Firefox better. If you are reporting a defect, please complete the following:
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Firefox/78.0
Build ID: 20200601093812
What were you doing?
Please tell us what site you were on, and what steps led to the error you are reporting
- Hamburger menu >
Customize…
. - Drag and drop Profiler icon into Overflow Menu.
What happened?
What, if any, error messages did you see? Did the browser crash? Screenshots or screen recordings can be attached to this report and are helpful in isolating and replicating the bug.
Please refer to attached screenshot.
When profiler is not recording:
Icon location is flipped to the right side. Icon itself is mirrored as expected.
text is mirrored and placed next to the wrongly placed icon.
The icon and text return to their expected location when a profile is being recorded.
What should have happened?
Was was the expected result? ("Not crashing" may be a valid answer.)
Icon is placed at the left, aligning with other icons.
Text is not mirrored.
Anything else we should know?
Do you have extensions installed? You can also go to about:support
in another window or tab and attach the report it generates to this bug.
My system and Nightly are en-US (LTR). I didn't test on a RTL locale.
Last good Nightly: 2020-05-27
First bad Nightly: 2020-05-28
pushlog: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=50d550c87f07e3fec8e2128f69b749a07b2cf18f&tochange=173e2e6757ae7c0f3c7b48836c7aea01bd5ba33c
Bisecting autoland builds:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=3a6dbe946bf20f93493b355889c6483075a9bdc6&tochange=60cf41955ab68d1cbfc78c961874bd79eb8bd476
It is regressed by bug 1634448.
Comment 2•4 years ago
|
||
Thanks for the report! My patch in bug 1621015 will fix this.
Assignee | ||
Comment 3•4 years ago
|
||
Updated•4 years ago
|
Assignee | ||
Comment 4•4 years ago
|
||
I'm submitting this as a separate patch as I plan on requesting for uplift.
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Comment 8•4 years ago
|
||
bugherder |
Comment 9•4 years ago
|
||
Were you still planning to nominate this for uplift? We have one beta left this cycle before 78 goes to RC.
Assignee | ||
Comment 10•4 years ago
|
||
Comment on attachment 9153958 [details]
Bug 1642330 - Fix profiler icon in the overflow menu; r?julienw
Beta/Release Uplift Approval Request
- User impact if declined: The profiler menu button, a fairly low usage feature would show up backwards in the overflow menu.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): This feature has fairly low usage, and is mostly used internally. The tool is still usable, just flipped the wrong way.
- String changes made/needed:
Comment 12•4 years ago
|
||
Comment on attachment 9153958 [details]
Bug 1642330 - Fix profiler icon in the overflow menu; r?julienw
approved for 78.0b9
Comment 13•4 years ago
|
||
bugherder uplift |
Updated•4 years ago
|
Updated•4 years ago
|
Comment 14•4 years ago
|
||
Reproduce the issue on Firefox 78.0a1 (2020-05-28) (20200528032513) on Windows 10 and Ubuntu 18.04.4 .
Verified fixed on Firefox 79.0a1 (2020-06-18) (20200618212828).
Comment 15•4 years ago
•
|
||
Verified fixed on Firefox beta dev edition 78.0b9 (20200619002543) on Windows 10 and Ubuntu 18.04.4
In Beta 78, the Profiler option is not displayed under Customize options. Should this be verified on Beta78? Is there any config parameter that should be updated so that the option be available also on Beta 78?
Assignee | ||
Comment 16•4 years ago
|
||
To turn it on in Beta:
- Go to https://profiler.firefox.com/
- Click "Enable profiler menu button"
Comment 17•4 years ago
•
|
||
Verified fixed on Firefox beta 78.0 (20200625152958) on Windows 10 and Ubuntu 18.04.4.
Description
•