joomla-cms icon indicating copy to clipboard operation
joomla-cms copied to clipboard

[5.1] Remove template special color

Open wilsonge opened this issue 2 years ago • 3 comments

Summary of Changes

Removes the template special color from core. The only potential place this is used is in toolbar secondary buttons. As far as I can tell no core extensions actually use this. But there is a chance third parties do. We now revert back to using the default bootstrap secondary color in this case.

This is a b/c break if other people are using this CSS Var. However I can't see any obvious way for us to actually deprecate CSS variables.

Testing Instructions

Change the class of an existing button from btn-*** (success/error/action) etc. to btn-secondary.

Link to documentations

Please select:

  • [ ] Documentation link for docs.joomla.org:

  • [x] No documentation changes for docs.joomla.org needed

  • [ ] Pull Request link for manual.joomla.org:

  • [x] No documentation changes for manual.joomla.org needed

wilsonge avatar Oct 01 '23 22:10 wilsonge

Have tested this successfully.

Cybernary avatar Oct 05 '23 18:10 Cybernary

@wilsonge deprecation is at least needed in the manual. is it critical to remove it in 5.0 or 5.x?

HLeithner avatar Oct 07 '23 09:10 HLeithner

Tested this but it seems that this has already been resolved.

For patch test patch test before installed after patch test patch test_path installed Removed again after patch patch test patch revert

webmasterab avatar Jun 09 '24 20:06 webmasterab

This pull request has been automatically rebased to 5.2-dev.

HLeithner avatar Sep 02 '24 08:09 HLeithner

@wilsonge could you update the PR to solve the conflicts?

Hackwar avatar Nov 22 '24 10:11 Hackwar

@wilsonge Any news about this one?

Hackwar avatar Jan 16 '25 08:01 Hackwar

I have solved ther conflicts this needs to be tested

rdeutz avatar Jan 18 '25 15:01 rdeutz

Tested, btn-secondary resolved to same colour before and after patch. Special color gone from Atum options.

No 'result selection' options in https://issues.joomla.org/tracker/joomla-cms/42017

Bodge-IT avatar Jan 19 '25 15:01 Bodge-IT

@wilsonge Can you please have a look at this and see if it's still relevant

Bodge-IT avatar Feb 19 '25 08:02 Bodge-IT