Skip to content

Tweak the space before the stack implementation filter's label#4581

Merged
julienw merged 2 commits into
firefox-devtools:mainfrom
julienw:tweak-space-filter-frames
Apr 21, 2023
Merged

Tweak the space before the stack implementation filter's label#4581
julienw merged 2 commits into
firefox-devtools:mainfrom
julienw:tweak-space-filter-frames

Conversation

@julienw

@julienw julienw commented Apr 19, 2023

Copy link
Copy Markdown
Contributor

Fixes #4580

Before:
image

After:
image

@julienw julienw requested review from canova and mstange April 19, 2023 09:45

@canova canova left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good to me, thanks! But I'm very bad at seeing these small differences for some reason. I couldn't see this on my previous review as well 😅 I think Markus is a lot better at this 😄

While I was testing it, I found another issue though. When I try to change the width and narrow down the Firefox window, the UI breaks at some point. See the screenshots below:

Screenshot 2023-04-21 at 12 06 58 PM

This is how it looks for flame graph:

Screenshot 2023-04-21 at 12 06 46 PM

I don't know if people actually use the profiler UI that narrow, but could be good to fix it (as a follow-up ofc).

Edit: Filed #4585 for that.

@julienw

julienw commented Apr 21, 2023

Copy link
Copy Markdown
Contributor Author

Thanks, I commented there!

@julienw julienw enabled auto-merge (squash) April 21, 2023 14:15
@julienw julienw merged commit 7dc6498 into firefox-devtools:main Apr 21, 2023
@canova canova mentioned this pull request May 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Imbalanced spacing in marker chart stack radio button row

2 participants