? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
29 Apr 2021

The icon used for featured/default is a yellow five pointed star
The icon used for non-featured/nondefault is a mono five pointed star

This fails basic accessibility as it is using color alone as an indicator

This PR changes the nonfeatured/nondefaault icon to a circle (this was already used in com_menus for non-home/non-default

To test apply pr and then

  • check the icon used for featured in com_content, com_contact
  • check the icon used for default in template styles
  • check the icon used for default in com_workflows

Pull Request for Issue #33296

After

image

avatar brianteeman brianteeman - open - 29 Apr 2021
avatar brianteeman brianteeman - change - 29 Apr 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 29 Apr 2021
Category Libraries
avatar Quy
Quy - comment - 29 Apr 2021

Contacts is still the mono five pointed star.

avatar richard67
richard67 - comment - 29 Apr 2021

Does this solve all aspects of the issue, or is there something remaining?

avatar alikon alikon - test_item - 30 Apr 2021 - Tested successfully
avatar alikon
alikon - comment - 30 Apr 2021

I have tested this item successfully on 5d3a50d


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

avatar sandramay0905 sandramay0905 - test_item - 30 Apr 2021 - Tested unsuccessfully
avatar sandramay0905
sandramay0905 - comment - 30 Apr 2021

I have tested this item ? unsuccessfully on 5d3a50d

As @Quy comment contacts show still the mono five pointed star.


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

avatar brianteeman brianteeman - change - 30 Apr 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 30 Apr 2021
Category Libraries Administration com_contact com_content Libraries
avatar brianteeman
brianteeman - comment - 30 Apr 2021

Should be correct now.

avatar richard67
richard67 - comment - 30 Apr 2021

@brianteeman Does this PR here solve issue #33296 completely? To me it seems it doesn't, because the list of issues in the issue's description seems to mention more than the description of this PR. And you had not closed the issue after making this PR, so I thought that could be by purpose. Was it right that @alikon closed the issue, or not?

avatar PhilETaylor PhilETaylor - test_item - 30 Apr 2021 - Tested successfully
avatar PhilETaylor
PhilETaylor - comment - 30 Apr 2021

I have tested this item successfully on c9d3265

tested icon used for featured in com_content, com_contact
tested icon used for default in template styles
tested icon used for default in com_workflows


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

avatar himanshu007-creator himanshu007-creator - test_item - 30 Apr 2021 - Tested successfully
avatar himanshu007-creator
himanshu007-creator - comment - 30 Apr 2021

I have tested this item successfully on c9d3265


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

avatar brianteeman
brianteeman - comment - 30 Apr 2021

@richard67 you are correct. I should have written partial pull request

avatar richard67 richard67 - change - 30 Apr 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 30 Apr 2021

RTC


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

avatar rdeutz rdeutz - close - 30 Apr 2021
avatar rdeutz rdeutz - merge - 30 Apr 2021
avatar rdeutz rdeutz - change - 30 Apr 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-04-30 11:09:46
Closed_By rdeutz
Labels Added: ?
avatar brianteeman
brianteeman - comment - 30 Apr 2021

thx

Add a Comment

Login with GitHub to post a comment