? Pending

User tests: Successful: Unsuccessful:

avatar puneeth2001
puneeth2001
18 Feb 2019

Pull Request for Issue #23863

Summary of Changes

Changed the color of field switcher labels.

Testing Instructions

Expected result

Actual result

Documentation Changes Required

avatar puneeth2001 puneeth2001 - open - 18 Feb 2019
avatar puneeth2001 puneeth2001 - change - 18 Feb 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Feb 2019
Category Repository
avatar wojsmol
wojsmol - comment - 18 Feb 2019

@puneeth2001 Please resolve merge conflict.

avatar C-Lodder
C-Lodder - comment - 18 Feb 2019

We don't use black font colours by default incase they're ever mixed with a white background which poses an a11y issue. Please use var(--dark) instead.

Please also add [4.0] to the title so everyone knows it's for Joomla 4

avatar puneeth2001 puneeth2001 - change - 18 Feb 2019
Labels Added: ?
avatar C-Lodder
C-Lodder - comment - 18 Feb 2019

@puneeth2001 still incorrect. Replace color: var(--dark, $gray-600); with color: var(--dark);

You also have a merge conflict which needs fixing.

avatar puneeth2001 puneeth2001 - change - 18 Feb 2019
Title
switcher lables color changed
[4.0] switcher lables color changed
avatar puneeth2001 puneeth2001 - edited - 18 Feb 2019
avatar puneeth2001 puneeth2001 - change - 18 Feb 2019
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2019-02-18 08:59:47
Closed_By puneeth2001
avatar puneeth2001 puneeth2001 - close - 18 Feb 2019
avatar Puneet-Jain-18
Puneet-Jain-18 - comment - 17 Apr 2019

Has'nt issue #23863 solved by this PR?? as the changes have been applied to the code but issue is still open.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 17 Apr 2019
avatar wilsonge
wilsonge - comment - 18 Apr 2019

I think so

Add a Comment

Login with GitHub to post a comment