-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WebProfilerBundle] Move panel specific styles to each panel #48757
Conversation
src/Symfony/Bundle/WebProfilerBundle/Resources/views/Profiler/profiler.css.twig
Outdated
Show resolved
Hide resolved
margin-bottom: 0; | ||
padding: 0; | ||
} | ||
#collector-content .sf-validator .trace li.selected { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Those styles are almost a copy/paste of the serializer and messenger ones (except the panel selector). This makes me think it might deserve a reusable component in the main styles instead of re-creating this rendering for each panel that has such a structure (and third-party bundles might also have a similar panel btw and would benefit from it)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK; but let's do it in a separate PR, please. Thanks.
c81478d
to
c43cfdf
Compare
@javiereguiluz Can you resolve the conflicts? |
b31d47f
to
507f2d7
Compare
I've rebased this PR. Thanks. |
Thank you @javiereguiluz. |
507f2d7
to
4c2375e
Compare
As suggested by @stof during the last profiler redesign, we should move specific panel styles to each panel, to ease the maintenance of the main CSS file.