? NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
16 Jan 2022

alternate to #36609

Some flags have a white background and so they do not display correctly. This is particularly true for the japanese flag but its true for other flags as well. This adds super thin border to flags with a white edge

Before

image

After

image

As this is a media file change it needs npm i

lets not start another debat here about not using flags or not usings gifs

avatar brianteeman brianteeman - open - 16 Jan 2022
avatar brianteeman brianteeman - change - 16 Jan 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 16 Jan 2022
Category Repository NPM Change
avatar brianteeman brianteeman - change - 16 Jan 2022
Labels Added: NPM Resource Changed ?
avatar Shubhamverma2796
Shubhamverma2796 - comment - 16 Jan 2022

I have tested this item successfully on 8a79b09


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

avatar Shubhamverma2796 Shubhamverma2796 - test_item - 16 Jan 2022 - Tested successfully
avatar RickR2H
RickR2H - comment - 18 Jan 2022

I have tested this item successfully on 8a79b09


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

avatar RickR2H RickR2H - test_item - 18 Jan 2022 - Tested successfully
avatar RickR2H RickR2H - change - 18 Jan 2022
Status Pending Ready to Commit
avatar RickR2H
RickR2H - comment - 18 Jan 2022

RTC


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

avatar bembelimen bembelimen - change - 20 Jan 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-01-20 03:02:18
Closed_By bembelimen
Labels Added: ?
avatar bembelimen bembelimen - close - 20 Jan 2022
avatar bembelimen bembelimen - merge - 20 Jan 2022
avatar bembelimen
bembelimen - comment - 20 Jan 2022

I guess we have the same problem with all the other flags if a color of the flag is used as background colour of the template (e.g. in frontend with the language switcher). So probably, we should update all flags sometimes in the future.

So far to fix the obvious ones this is a good start. Thx.

avatar brianteeman
brianteeman - comment - 20 Jan 2022

Thanks for merging.

Now that all the flags have a solid border then css would be a better solution if there is a dark background colour that clashes.

Add a Comment

Login with GitHub to post a comment