Feeling Lucky
?
[#3820] - [#33869] [imp] Improve the "Is debug" check for the templates
- Closed
- 8 Aug 2014
- Medium
- Build: staging
- # 3820
- Diff
- zero-24:patch-36
- Foreign ID 33869
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Tracker
How to test
- enable debug
- make sure that the debug css is load if debug is on for all templates.
| Title |
|
||||||
zero-24
- comment
- 14 Jul 2014
Patch seems more code-styling changes rather then a fix.
Sure 
But after the change it is easier and smaler code. Sure it is more a "code style" PR (make our code more easy to understand and maintain) but it change the code so it need testing.
But as you note. Nothing is broken bevor or after the PR.
| Status | New | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-08-08 15:27:51 |
@test unsuccesful
The debug.css is already loading for all templates before applying the patch. Patch seems more code-styling changes rather then a fix.
Maybe @zero-24 can provide better test instructions?