Skip to content

Conversation

@e-moe
Copy link
Contributor

@e-moe e-moe commented Jun 20, 2017

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

@e-moe e-moe changed the title [WebProfilerBundle] Eliminate line wrap on count columnt (routing) [WebProfilerBundle] Eliminate line wrap on count column (routing) Jun 20, 2017
@javiereguiluz
Copy link
Member

Thank you @e-moe.

@javiereguiluz javiereguiluz merged commit 78f1fde into symfony:2.8 Jun 20, 2017
javiereguiluz added a commit that referenced this pull request Jun 20, 2017
…outing) (e-moe)

This PR was merged into the 2.8 branch.

Discussion
----------

[WebProfilerBundle] Eliminate line wrap on count column (routing)

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

Commits
-------

78f1fde [WebProfilerBundle] Eliminate line wrap on count columnt (routing)
@stof
Copy link
Member

stof commented Jun 20, 2017

does it also affect other similar panels ?

@e-moe
Copy link
Contributor Author

e-moe commented Jun 20, 2017

I briefly checked other profiler panes/pages - they already using nowrap class in similar places

This was referenced Jul 4, 2017
ostrolucky pushed a commit to ostrolucky/symfony that referenced this pull request Mar 25, 2018
…lumn (routing) (e-moe)

This PR was merged into the 2.8 branch.

Discussion
----------

[WebProfilerBundle] Eliminate line wrap on count column (routing)

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

Commits
-------

78f1fde [WebProfilerBundle] Eliminate line wrap on count columnt (routing)
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