Commit e363f16
committed
bug symfony#59229 [WebProfilerBundle] fix loading of toolbar stylesheet (alexislefebvre)
This PR was merged into the 7.2 branch.
Discussion
----------
[WebProfilerBundle] fix loading of toolbar stylesheet
| Q | A
| ------------- | ---
| Branch? | 7.2
| Bug fix? | yes
| New feature? | no
| Deprecations? | no
| Issues | Fix symfony#59045
| License | MIT
It looks like this PR
- symfony#58287
Caused issues with some configurations:
- symfony#59045
According to the thumb-up emoji on [this comment](symfony#59045 (comment)) (I don’t have a better measurement of the impact), it affected at least 10 users, with various web servers.
Proposals:
1. do not use the `.css` file extension so that servers do not try to serve an actual file
2. if we consider that the disappearance of the style of the profiler’s toolbar is a breaking change, the `.css` file extension could be added back with Symfony 8.0, with a note to help people upgrade (see the workarounds in the issue)
Commits
-------
7fef930 fix: loading of WebProfilerBundle’s toolbar stylesheetFile tree
2 files changed
+7
-7
lines changed- src/Symfony/Bundle/WebProfilerBundle
- Resources/config/routing
- Tests/Controller
2 files changed
+7
-7
lines changedLines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
| 7 | + | |
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
| |||
Lines changed: 6 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
152 | 152 | | |
153 | 153 | | |
154 | 154 | | |
155 | | - | |
156 | | - | |
157 | | - | |
| 155 | + | |
| 156 | + | |
158 | 157 | | |
159 | | - | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
160 | 161 | | |
161 | | - | |
162 | 162 | | |
163 | | - | |
| 163 | + | |
164 | 164 | | |
165 | 165 | | |
166 | 166 | | |
| |||
0 commit comments