Skip to content

[WebProfilerBundle] Fix Image/SVG display in WebProfiler toolbar. #25433

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

Merged
merged 1 commit into from
Dec 11, 2017
Merged

[WebProfilerBundle] Fix Image/SVG display in WebProfiler toolbar. #25433

merged 1 commit into from
Dec 11, 2017

Conversation

bobdenotter
Copy link
Contributor

Q A
Branch? 2.7
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

This small fix prevents widths from "creeping in" from previously defined CSS rules into the WebProfiler toolbar. See before/after screenshot:

screen_shot_2017-12-11_at_11_56_06

Note: This PR fixes an other issue than #18469, which landed in SF 2.8.

Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

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

Nice fix. Have you tried it in different popular browsers to be sure there are no side effects? Thanks!

@bobdenotter
Copy link
Contributor Author

Hi @javiereguiluz, good question.

I confirmed the issue on Chrome, Firefox, Safari and MS Edge across OSX and Win10. This fix works on all tested browsers.

Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this and for testing it in all browsers!

@javiereguiluz javiereguiluz merged commit 62f5eae into symfony:2.7 Dec 11, 2017
javiereguiluz added a commit that referenced this pull request Dec 11, 2017
… toolbar. (bobdenotter)

This PR was merged into the 2.7 branch.

Discussion
----------

[WebProfilerBundle] Fix Image/SVG display in WebProfiler toolbar.

| Q             | A
| ------------- | ---
| Branch?       | 2.7
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

This small fix prevents widths from "creeping in" from previously defined CSS rules into the WebProfiler toolbar. See before/after screenshot:

![screen_shot_2017-12-11_at_11_56_06](https://user-images.githubusercontent.com/1833361/33828617-1e59c2c2-de6d-11e7-8c5e-560eee847205.png)

Note: This PR fixes an other issue than #18469, which landed in SF 2.8.

Commits
-------

62f5eae Set `width: auto` on WebProfiler toolbar's reset.
@bobdenotter bobdenotter deleted the patch-1 branch December 11, 2017 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants