? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
31 Mar 2021

Some of the badges are set to have a border that is a color lighter than the badge text. Unfortunately that doesn't work if the text is white you get a white border.

All the badges need to have a visible border otherwise the badges will appear visually to be of different sizes.

This PR changes the border of the two existing white bordered badges to use the same shade of the dark text that the bg-info badge uses.

Testers dont forget to run npm to rebuild the css

The easies way to to check this would be in the article list after enabling the voting plugin

before

image

after

image

avatar brianteeman brianteeman - open - 31 Mar 2021
avatar brianteeman brianteeman - change - 31 Mar 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 31 Mar 2021
Category Administration Templates (admin) NPM Change
avatar ceford ceford - test_item - 1 Apr 2021 - Tested successfully
avatar ceford
ceford - comment - 1 Apr 2021

I have tested this item successfully on 41fb0e8


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

avatar Quy
Quy - comment - 1 Apr 2021

Wouldn't it better to remove the border? Why override Bootsrap badges when they look the same except for the border?

avatar meinhoonharsh meinhoonharsh - test_item - 1 Apr 2021 - Tested successfully
avatar meinhoonharsh
meinhoonharsh - comment - 1 Apr 2021

I have tested this item successfully on 41fb0e8

## Before:
image

After:

image


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32951.
avatar brianteeman
brianteeman - comment - 1 Apr 2021

Wouldn't it better to remove the border? Why override Bootsrap badges when they look the same except for the border?

I am not redesigning the template I am just fixing an obvious bug. From the code we can see that the intention was clearly to have a border but it ended up invisible

avatar Quy Quy - change - 1 Apr 2021
Status Pending Ready to Commit
Labels Added: ?
avatar Quy
Quy - comment - 1 Apr 2021

RTC


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

avatar Quy Quy - change - 1 Apr 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-04-01 17:17:42
Closed_By Quy
Labels Added: ?
avatar Quy Quy - close - 1 Apr 2021
avatar Quy Quy - merge - 1 Apr 2021
avatar Quy
Quy - comment - 1 Apr 2021

Thanks.

avatar brianteeman
brianteeman - comment - 1 Apr 2021

thanks

Add a Comment

Login with GitHub to post a comment