[WebProfiler] Sidebar hover bug on small window #18405

norkunas · 2016-04-01T12:59:00Z

Browser Chrome 49.0.2623.110
OS Windows
SF 2.8

javiereguiluz · 2016-04-01T13:26:36Z

I can't reproduce it:

The sidebar menu should collapse automatically on small screens. This feature was implemented by @hason in #15818.

norkunas · 2016-04-01T13:56:39Z

@javiereguiluz you didn't hover on sidebar

norkunas · 2016-04-01T14:00:24Z

Also looking at your gif my eye caught this

So if you hover on the sidebar i think you'll see what i see

javiereguiluz · 2016-04-01T14:11:04Z

@norkunas I'm sorry. You are right. This should be fixed in #18407.

…ereguiluz) This PR was submitted for the master branch but it was merged into the 2.8 branch instead (closes #18407). Discussion ---------- Fixed the "hover" state of the profiler sidebar menu | Q | A | ------------- | --- | Branch? | 2.8 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #18405 | License | MIT | Doc PR | - ### Before ![before_sidebar](https://cloud.githubusercontent.com/assets/73419/14209368/2cc6e168-f824-11e5-91e7-70fab8d64d12.gif) ### After ![after_sidebar](https://cloud.githubusercontent.com/assets/73419/14209376/3295d7a2-f824-11e5-94b5-254f1fc4e027.gif) Commits ------- 85deb30 Fixed the "hover" state of the profiler sidebar menu

javiereguiluz added Bug WebProfilerBundle Unconfirmed labels Apr 1, 2016

carsonbot added the Status: Needs Review label Apr 1, 2016

javiereguiluz mentioned this issue Apr 1, 2016

Fixed the "hover" state of the profiler sidebar menu #18407

Closed

javiereguiluz added Status: Reviewed and removed Unconfirmed Status: Needs Review labels Apr 1, 2016

fabpot closed this as completed Apr 1, 2016

Provide feedback