NPM Resource Changed ? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
17 Jun 2020

scss has been changed in this pr so you will need to node build.js --compile-css

The problem

The toolbar buttons in the media manager modal eg when adding an intro image is displayed with the full colours and no spacing

The solution

This could be technically wrong. It works but maybe there is a better way. It wraps the toolbar with the subhead div and adds a noshadow class

before

image

after

image

avatar brianteeman brianteeman - open - 17 Jun 2020
avatar brianteeman brianteeman - change - 17 Jun 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Jun 2020
Category Administration com_media Templates (admin) NPM Change
avatar richard67 richard67 - test_item - 18 Jun 2020 - Tested successfully
avatar richard67
richard67 - comment - 18 Jun 2020

I have tested this item successfully on f635694

No idea if it is technically the best solution or not. Maybe @ciar4n can tell, or @C-Lodder ?

But it works well.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/29681.

avatar C-Lodder C-Lodder - test_item - 18 Jun 2020 - Tested successfully
avatar C-Lodder
C-Lodder - comment - 18 Jun 2020

I have tested this item successfully on f635694

This'll do


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/29681.

avatar richard67 richard67 - change - 18 Jun 2020
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 18 Jun 2020

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/29681.

avatar richard67 richard67 - change - 18 Jun 2020
Labels Added: ? ? NPM Resource Changed
avatar brianteeman
brianteeman - comment - 18 Jun 2020

Thank @C-Lodder I wasn't sure at all if it was technically correct

avatar brianteeman brianteeman - change - 18 Jun 2020
Labels
avatar Quy Quy - change - 18 Jun 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-06-18 17:05:04
Closed_By Quy
avatar Quy Quy - close - 18 Jun 2020
avatar Quy Quy - merge - 18 Jun 2020
avatar brianteeman
brianteeman - comment - 18 Jun 2020

thanks

Add a Comment

Login with GitHub to post a comment