Skip to content

[WebProfilerBundle] Added table-layout property to AJAX toolbar css #18237

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
Mar 21, 2016
Merged

[WebProfilerBundle] Added table-layout property to AJAX toolbar css #18237

merged 1 commit into from
Mar 21, 2016

Conversation

kevintweber
Copy link

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

When a page contains the following css:

table {
    table-layout: fixed;
}

Then the AJAX panel has broken table cell spacing. All the cells are mostly on top of each other.

This patch fixes that situation by overriding the table-layout property for the AJAX panel.

@javiereguiluz
Copy link
Member

👍

@kevintweber thanks for this fix.

@fabpot
Copy link
Member

fabpot commented Mar 21, 2016

Thank you @kevintweber.

@fabpot fabpot merged commit 0fd9503 into symfony:2.8 Mar 21, 2016
fabpot added a commit that referenced this pull request Mar 21, 2016
…olbar css (kevintweber)

This PR was merged into the 2.8 branch.

Discussion
----------

[WebProfilerBundle] Added table-layout property to AJAX toolbar css

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

When a page contains the following css:
```css
table {
    table-layout: fixed;
}
```
Then the AJAX panel has broken table cell spacing.  All the cells are mostly on top of each other.

This patch fixes that situation by overriding the table-layout property for the AJAX panel.

Commits
-------

0fd9503 Added WebProfiler toolbar ajax panel table layout css.
This was referenced Mar 27, 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