Feeling Lucky
?
[#6200] - Bootstrapization of warning panels in com_installer
- Closed
- 7 Mar 2015
- Medium
- Build: staging
- Easy Test
- # 6200
- Diff
- peterlose:warnings
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Before PR
After PR
losedk
- 
joomla-cms-bot
- | Labels |
Added:
?
|
||
C-Lodder
- comment
- 27 Feb 2015
Nice one.
Just a thought. Would it not be better to use Bootstrap alert boxes instead of an accordion?
n9iels
- comment
- 27 Feb 2015
$i is not defined, it can be removed over here
zero-24
- comment
- 27 Feb 2015
@test @losedk works good here. Thanks! One more tester here 
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6200.
| Easy | No | ⇒ | Yes |
| Category | ⇒ | Administration UI/UX |
brianteeman
- | Category | Administration UI/UX | ⇒ | Administration Templates (admin) UI/UX |
n9iels
- comment
- 2 Mar 2015
@test works fine for me, thanks!
Two tests, moving to RTC!
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6200.
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
?
|
||
nonumber
- comment
- 4 Mar 2015
You are correct. My mistake.
| Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-03-07 04:05:01 |
phproberto
- comment
- 7 Mar 2015
Merged. Thanks!
@test works fine, good change!
Please add some code style and remove empty lines