Feeling Lucky
?
[#6356] - Remove old legacy-icons in com_content
- Closed
- 7 Mar 2015
- Medium
- Build: staging
- # 6356
- Diff
- n9iels:remove-legacy-icons
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
This PR is a follow up of for #6248. I looked at the code again and I concluded the following:
- There is currently no parameter for enable/disable the legacy icons
- None of the legacy icons (except the checkout icon I fixed) contains text next to the icon
So I had two chooses, repair them or remove them. I choose for the last option because I think there or no people who are using this icons or even know they exist.
How to test this patch
- Enable the following option in the configuration of Article manager
- Show Icons
- Show Print Icon
- Show Email Icon
- Go to the front-end and confirm all these icons are load under the cog-button
- Login on the front-end
- Confirm the edit icon is also load under the cog-button
joomla-cms-bot
- | Labels |
Added:
?
|
||
n9iels
- comment
- 7 Mar 2015
oeps.. 
Well that makes clear why there is no text behind the icon.
So I close this PR because in favour of the beez3 template, removing these icons is impossible.
Thanks for noticing!
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-03-07 07:48:55 |
This prevents Beez3 from loading the icons.
This was the original reason behind using legacy.