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
[Hackday] [Toolbar] Fix info position and icons on small screens #12649
Conversation
Here is a sample app to test the changes, thanks to Heroku ;) |
wow great PR experience, screenshots, heroku app to review it, thanks 👍 |
@@ -11,7 +11,7 @@ | |||
</service> | |||
|
|||
<service id="data_collector.dump" class="Symfony\Component\HttpKernel\DataCollector\DumpDataCollector"> | |||
<tag name="data_collector" id="dump" template="@Debug/Profiler/dump.html.twig" /> | |||
<tag name="data_collector" id="dump" template="@Debug/Profiler/dump.html.twig" priority="285" /> |
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.
Why did you change all priorities?
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.
Sorry, I forgot to mention in the PR description. Basically only to allow more flexibility if a user wants to put something custom between collectors, but that's not strictly needed.
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.
Can you revert the priority changes?
Place info div to not float out of the window
c00ab5d
to
23c0ddc
Compare
@fabpot done! |
According to your sample app, there is some issues where some panels are getting smaller width than what they should on hover, making it impossible to read the content (look at the dump panel for instance) |
however, allowing icons to go on several lines when there is not enough space on 1 line to display the whole toolbar is a simple way to fix the issue of items disappearing |
@stof thanks, it wasn't easy, but I spotted the bug :) |
caf4506
to
838857c
Compare
👍 |
Thank you @stefanosala. |
…creens (stefanosala) This PR was merged into the 2.6 branch. Discussion ---------- [Hackday] [Toolbar] Fix info position and icons on small screens | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #12582 | License | MIT | Doc PR | - I changed a bit the order of profiles, IMO the dump goes closer to request data. Then I tweaked a bit the toolbar to show more icons even on smaller screen. I tested everything with Chrome and IE9 and it seems to work smoothly. Here are some explanatory screenshots: ![](http://cl.ly/image/3E2e360Y0a15/Screen%20Shot%202014-11-29%20at%2011.56.17.png) ![](http://cl.ly/image/1d073v2v2U2J/Screen%20Shot%202014-11-29%20at%2011.56.42.png) Commits ------- 838857c [Toolbar] Reset right and left .sf-toolbar-info values needed for window resize 23c0ddc [Toolbar] Handle info position via javascript Place info div to not float out of the window 7a8582e [Toolbar] Display all icons, even on two lines for small screens
…small screens (stefanosala) This PR was merged into the 2.6 branch. Discussion ---------- [Hackday] [Toolbar] Fix info position and icons on small screens | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | symfony#12582 | License | MIT | Doc PR | - I changed a bit the order of profiles, IMO the dump goes closer to request data. Then I tweaked a bit the toolbar to show more icons even on smaller screen. I tested everything with Chrome and IE9 and it seems to work smoothly. Here are some explanatory screenshots: ![](http://cl.ly/image/3E2e360Y0a15/Screen%20Shot%202014-11-29%20at%2011.56.17.png) ![](http://cl.ly/image/1d073v2v2U2J/Screen%20Shot%202014-11-29%20at%2011.56.42.png) Commits ------- 838857c [Toolbar] Reset right and left .sf-toolbar-info values needed for window resize 23c0ddc [Toolbar] Handle info position via javascript Place info div to not float out of the window 7a8582e [Toolbar] Display all icons, even on two lines for small screens
I changed a bit the order of profiles, IMO the dump goes closer to request data. Then I tweaked a bit the toolbar to show more icons even on smaller screen. I tested everything with Chrome and IE9 and it seems to work smoothly.
Here are some explanatory screenshots: