Closed Bug 1284433 Opened 8 years ago Closed 2 years ago

Don't overlap checkbox and marker in performance-filter-menupopup

Categories

(DevTools :: Performance Tools (Profiler/Timeline), defect, P2)

All
Windows
defect

Tracking

(firefox-esr52 wontfix)

RESOLVED WONTFIX
Tracking Status
firefox-esr52 --- wontfix

People

(Reporter: magicp.jp, Unassigned)

References

Details

(Keywords: regression, Whiteboard: [good taipei bug][lang=css])

Attachments

(1 file)

Attached image performance-filter-menupopup.png (deleted) —
User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:50.0) Gecko/20100101 Firefox/50.0 Build ID: 20160704030211 Steps to reproduce: 1. Start Nightly in Windows 2. Open DevTools > Performance 3. Open the performance filter Actual results: checkbox and marker are overlapped. Regression range: https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=975d97ff77ae82b113b66d2a46dc30fc654f411a&tochange=c0edf384327531b1e302c61fa3c77cf966152182 Expected results: Don't overlap checkbox and marker.
Blocks: 1172412
Has Regression Range: --- → yes
Has STR: --- → yes
Component: Untriaged → Developer Tools: Performance Tools (Profiler/Timeline)
OS: Unspecified → Windows
Hardware: Unspecified → All
Priority: -- → P1
One thing to try is adding start padding on .menu-iconic-text then using a background-image to emulate the square (probably with a solid linear-gradient and a fixed background size and position).
Mentor: ntim.bugs
Whiteboard: [good taipei bug][lang=css]
Looks good on latest nightly Version 51.0a1 Build ID 20160810150918 User Agent Mozilla/5.0 (X11; Linux x86_64; rv:51.0) Gecko/20100101 Firefox/51.0 OS Linux 4.2.0-42-generic Can anyone verify ??
Flags: needinfo?(ntim.bugs)
(In reply to [:Towkir] Ahmed from comment #2) > Looks good on latest nightly > > Version 51.0a1 > Build ID 20160810150918 > User Agent Mozilla/5.0 (X11; Linux x86_64; rv:51.0) Gecko/20100101 > Firefox/51.0 > OS Linux 4.2.0-42-generic > > Can anyone verify ?? The issue is on Windows.
Flags: needinfo?(ntim.bugs) → needinfo?(3ugzilla)
Ok, but is there different styling for different platform ? I mean different performance.css files ?
Flags: needinfo?(3ugzilla)
I have reproduced this bug with Nightly 50.0a1 (2016-07-05) Build ID 20160705030222 User Agent Mozilla/5.0 (Windows NT 6.3; Win64; x64; rv:50.0) Gecko/20100101 Firefox/50.0
Blocks: perf-bug
As per the DevTools triage process doc [1], a P1 is defined as: - a security hole - causing data-loss - causing a crash - stopping the tools working - embarrassing Should be fixed within 24h and your manager should know about it and, between you, assign it to someone who can fix it. This bug has been set to P1 more than 3 months ago and is still unassigned. I don't think it's that bad that it needs to be a P1. [1] https://docs.google.com/document/d/1uG0foc0pphXJB489_8ClKjYr1wbRXeFyrkd_kIOv9ao/edit#
Priority: P1 → P2
Mass wontfix for bugs affecting firefox 52.
Hi I am interested in taking this bug. Please let me know if i can take this one or suggest other bugs as I am a newbie.
Product: Firefox → DevTools
Hi I am interested in taking this bug.
Mentor: ntim.bugs
Hey Irina, sure, please do! I added you as the assignee for this bug. The code for the performance tool is in /devtools/client/performance [1], but maybe the filter component is shared elsewhere. You can also find a lot of documentation about how to develop in the devtools codebase in [2]. I encourage you to follow the steps outlined in this documentation. I don't know the codebase well enough to give you directions, but I can have a look if you need it. Please add a "needinfo" request if you need anything. [1] https://searchfox.org/mozilla-central/source/devtools/client/performance [2] https://docs.firefox-dev.tools/ Thanks again for your help !
Assignee: nobody → irina6793
Assignee: irina6793 → nobody

Now that we replaced the old performance panel by the Firefox Profiler (see Bug 1668219) we're closing bugs related to this. If you see similar issues on the new profiler, please file another bug, and we'll be happy to have a look.

Filter on MASSCLOSEOLDPERFTOOLBUGS.

Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: