Skip to content

Fixed the antialiasing of the toolbar text #17860

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
Feb 20, 2016
Merged

Conversation

javiereguiluz
Copy link
Member

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

Problem

The Symfony toolbar looks ugly in some situations (the text is too thin and unreadable); for example inside a SonataAdmin backend:

(top: toolbar in a normal page; bottom: toolbar in Sonata backend; if you don't see any difference, click on the image to zoom it)

Chrome

before_chrome

Firefox

before_firefox

Solution

The cause of this problem are the styles applied by the popular AdminLTE template used by Sonata and other admin bundles. The proposed changes neutralize any aliasing set by external CSS styles, so the toolbar is always displayed the same way:

Chrome

after_chrome

Firefox

after_firefox

@xabbuh
Copy link
Member

xabbuh commented Feb 20, 2016

👍

Status: Reviewed

@fabpot
Copy link
Member

fabpot commented Feb 20, 2016

Thank you @javiereguiluz.

@fabpot fabpot merged commit 5ad065d into symfony:2.8 Feb 20, 2016
fabpot added a commit that referenced this pull request Feb 20, 2016
This PR was merged into the 2.8 branch.

Discussion
----------

Fixed the antialiasing of the toolbar text

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

### Problem

The Symfony toolbar looks ugly in some situations (the text is too thin and unreadable); for example inside a SonataAdmin backend:

(top: toolbar in a normal page; bottom: toolbar in Sonata backend; if you don't see any difference, click on the image to zoom it)

#### Chrome

![before_chrome](https://cloud.githubusercontent.com/assets/73419/13181902/8836e586-d72f-11e5-9394-490f81978606.png)

#### Firefox

![before_firefox](https://cloud.githubusercontent.com/assets/73419/13181905/8cca5eac-d72f-11e5-9905-b85cb5249a45.png)

### Solution

The cause of this problem are the styles applied by the popular AdminLTE template used by Sonata and other admin bundles. The proposed changes neutralize any aliasing set by external CSS styles, so the toolbar is always displayed the same way:

#### Chrome

![after_chrome](https://cloud.githubusercontent.com/assets/73419/13182022/0e15fade-d730-11e5-8e0b-d5a712c7158e.png)

#### Firefox

![after_firefox](https://cloud.githubusercontent.com/assets/73419/13182024/104e3cb2-d730-11e5-83d8-86cb52b87358.png)

Commits
-------

5ad065d Fixed the antialiasing of the toolbar text
javiereguiluz added a commit to EasyCorp/EasyAdminBundle that referenced this pull request Feb 24, 2016
This PR was merged into the master branch.

Discussion
----------

Make the Symfony Debug toolbar look always nice

This is a port of symfony/symfony#17860 to fix how the Symfony Debug Toolbar looks inside EasyAdmin.

I want to include this fix in our CSS styles because that will make the Symfony toolbar look always nice inside EasyAdmin. Otherwise, only people with the latest Symfony versions will see it right.

Commits
-------

2c61ae7 Make the Symfony Debug toolbar look always nice
This was referenced Feb 28, 2016
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