?
avatar C-Lodder
C-Lodder
26 Mar 2020

Steps to reproduce the issue

When attempting to override the custom-select background colour, it's not being applied across the entire element. This is becuase the SVG isn't being used as only an icon, but instead a background for the entire element.

Expected result

The SVG should only consist of the arrow icon, allowing users to change the background colour of the dropdown.

Screenshot

screeny1

avatar C-Lodder C-Lodder - open - 26 Mar 2020
avatar joomla-cms-bot joomla-cms-bot - change - 26 Mar 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 26 Mar 2020
avatar infograf768
infograf768 - comment - 26 Mar 2020

+1 Having fought with these svg since the beginning of this template, I would love the idea of getting rid of them and go back to icons indeed.

avatar brianteeman
brianteeman - comment - 26 Mar 2020

you both keep saying this but all it needs is a pull request

avatar jwaisner jwaisner - change - 26 Mar 2020
Priority Medium Low
Status New Confirmed
Build staging 4.0-dev
avatar jwaisner jwaisner - change - 26 Mar 2020
Category Layout
avatar infograf768
infograf768 - comment - 27 Mar 2020

I would not know how to do that correctly.

avatar Quy
Quy - comment - 15 Apr 2020

@C-Lodder Please advise if PR #28680 is the correct fix. Thanks.

avatar C-Lodder
C-Lodder - comment - 15 Apr 2020

@Quy That PR only changes the contrast

avatar C-Lodder
C-Lodder - comment - 22 Apr 2020

@brianteeman & template team, are you happy with this?

screeny

If so, I'll submit the PR

avatar infograf768
infograf768 - comment - 22 Apr 2020

Except for the purple colour, I like this to get rid of the svg(s)
There are other places to modify though
Screen Shot 2020-04-22 at 11 23 06

@astridx @ciar4n

avatar C-Lodder
C-Lodder - comment - 22 Apr 2020

@infograf768 Ignore the purple colour of the text. That was just me testing the hue settings in the template.

avatar astridx
astridx - comment - 22 Apr 2020

@C-Lodder
Please submit the PR.

avatar C-Lodder
C-Lodder - comment - 22 Apr 2020
avatar C-Lodder C-Lodder - change - 22 Apr 2020
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2020-04-22 11:02:16
Closed_By C-Lodder
avatar C-Lodder C-Lodder - close - 22 Apr 2020

Add a Comment

Login with GitHub to post a comment