Feeling Lucky
?
[#8830] - Administrator main menu list improvements
- Closed
- 17 Jan 2016
- Medium
- Build: staging
- # 8830
- Diff
- andrepereiradasilva:menus-list-fix
- Success Human Test Results: 3 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Description
This PR does some improvements to administrator main menu list:
- corrects an html error (missing " between href and title)
- correct spaces between table lines -> (p to div)
- put counters items colors like in categories, banners, contacts components
- code style: adds ; and spaces at the end of php code
- some minor adjustments
Before patch:
After patch:
How to test
- Fresh Joomla install with multilanguage (several languages).
- Apply this patch
- Go to menus view
- Check if everything is ok.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
anibalsanchez
- comment
- 6 Jan 2016
I have tested this item 
successfully on 3462ea2
Test OK
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8830.
| Status | Pending | ⇒ | Ready to Commit |
brianteeman
- comment
- 6 Jan 2016
RTC - thanks
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8830.
| Labels |
Added:
?
|
||
andrepereiradasilva
- comment
- 6 Jan 2016
@justinherrin, @anibalsanchez thanks for testing
rdeutz
- | Milestone |
Added: |
||
NLRoosj
- comment
- 10 Jan 2016
I have tested this item successfully on 3462ea2
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8830.
| Milestone |
Added: |
||
| Milestone |
Removed: |
||
| Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-01-17 14:56:31 |
| Closed_By | ⇒ | andrepereiradasilva |
| Labels |
Removed:
?
|
||
wilsonge
- comment
- 17 Jan 2016
Thanks!
I have tested this item
successfully on 3462ea2
Tested PR successfully. Things look okay.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8830.