Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[WebProfilerBundle ] Fix toolbar vertical alignment #11641

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

Closed
wants to merge 1 commit into from
Closed

[WebProfilerBundle ] Fix toolbar vertical alignment #11641

wants to merge 1 commit into from

Conversation

blaugueux
Copy link
Contributor

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

Fix the vertical alignment of texts in the toolbar.

This issue appears when a reset css contain vertical-align: top/bottom/middle;

@blaugueux blaugueux changed the title Fix toolbar vertical alignment. [WebProfilerBundle ] Fix toolbar vertical alignment Aug 11, 2014
@romainneutron
Copy link
Contributor

Could you paste a screenshot before/after ?

@blaugueux
Copy link
Contributor Author

Before:
image

After:
image

@stof
Copy link
Member

stof commented Aug 13, 2014

👍

@blaugueux
Copy link
Contributor Author

thx @stof

@nicolas-grekas
Copy link
Member

Could you please rebase this PR on branch 2.3? 👍 from me.

@stof
Copy link
Member

stof commented Aug 13, 2014

@nicolas-grekas why not just doing it with the GH tool when merging ? If @blaugueux does the rebase, it will also require recreating another PR to change the target.

@nicolas-grekas
Copy link
Member

Right! I'll do that

nicolas-grekas added a commit that referenced this pull request Aug 13, 2014
…ueux)

This PR was submitted for the master branch but it was merged into the 2.3 branch instead (closes #11641).

Discussion
----------

[WebProfilerBundle ] Fix toolbar vertical alignment

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

Fix the vertical alignment of texts in the toolbar.

This issue appears when a reset css contain ```vertical-align: top/bottom/middle;```

Commits
-------

65220e7 Fix toolbar vertical alignment.
@nicolas-grekas
Copy link
Member

Closed via 48b150a
Thank you @blaugueux

@blaugueux
Copy link
Contributor Author

Thx @nicolas-grekas :)

@blaugueux blaugueux deleted the profiler-toolbar-css branch August 13, 2014 19:50
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.

4 participants