? ? Pending

User tests: Successful: Unsuccessful:

avatar C-Lodder
C-Lodder
30 Dec 2017

Summary of Changes

This PR migrates the Switcher field over to a dedicated custom element which has been approved by the a11y team.

Testing Instructions

Go to the Global Configuration (or any other page that uses a switcher) and ensure it works the same as it did before.

Note, please ignore the colours. We still need to publish an update to NPM and properly pull the elements using the new package manager. Once done and we start working on the new template, we'll work the the necessary SCSS changes.

Hope @brianteeman likes the removal or the border radius :)

avatar C-Lodder C-Lodder - open - 30 Dec 2017
avatar C-Lodder C-Lodder - change - 30 Dec 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 30 Dec 2017
Category Administration Templates (admin) Layout JavaScript Front End Templates (site)
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 30 Dec 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Dec 2017

I have tested this item successfully on e455b9d


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

avatar dgt41
dgt41 - comment - 30 Dec 2017

@C-Lodder nice one! Please check either #19231 or #19184 for the source code folder. Basically you need to move the src to build/webcomponents/field-switcher and then run grunt createElements

avatar Anu1601CS Anu1601CS - test_item - 31 Dec 2017 - Tested successfully
avatar Anu1601CS
Anu1601CS - comment - 31 Dec 2017

I have tested this item successfully on e455b9d


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

avatar Anu1601CS Anu1601CS - test_item - 31 Dec 2017 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 31 Dec 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 31 Dec 2017

Ready to Commit after two successful tests.

avatar brianteeman
brianteeman - comment - 31 Dec 2017

Why is there always a "throbber" outline on the No value?

avatar brianteeman
brianteeman - comment - 31 Dec 2017

Ignore that comment - it was a cache issue

avatar brianteeman
brianteeman - comment - 31 Dec 2017

Nope it was a brian issue and is a problem - you can probably only see it when you tab from one switcher to another

avatar brianteeman
brianteeman - comment - 31 Dec 2017

(i will post a video soon - there are a few issues)

avatar C-Lodder
C-Lodder - comment - 31 Dec 2017

Brian, I purposely added the pulsating focus state, not an issue. Inspired by Google

avatar brianteeman
brianteeman - comment - 31 Dec 2017

But why is it always on the NO value

throbber-and-outline

avatar brianteeman
brianteeman - comment - 31 Dec 2017

ok - i get it now. its not on the no value but its on the switch

avatar brianteeman brianteeman - test_item - 31 Dec 2017 - Tested successfully
avatar brianteeman
brianteeman - comment - 31 Dec 2017

I have tested this item successfully on e455b9d


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

avatar C-Lodder C-Lodder - change - 2 Jan 2018
Labels Added: ?
avatar C-Lodder C-Lodder - change - 2 Jan 2018
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 2 Jan 2018
Category Administration Templates (admin) Layout JavaScript Front End Templates (site) JavaScript Repository Administration Templates (admin) Layout Front End Templates (site)
avatar wilsonge wilsonge - change - 3 Jan 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-01-03 00:05:15
Closed_By wilsonge
avatar wilsonge wilsonge - close - 3 Jan 2018
avatar wilsonge wilsonge - merge - 3 Jan 2018
avatar dgt41
dgt41 - comment - 3 Jan 2018

@C-Lodder the src needs to move to the right path build\...
and the compiled ones using build.js have to move to media\system\webcomponents\...

avatar C-Lodder
C-Lodder - comment - 3 Jan 2018

@dgt41 yeah dont worry I haven't forgotten. compileElements grunt task didn't work so I'm waiting on your PR

avatar dgt41
dgt41 - comment - 3 Jan 2018

Hold on tight 😂

Add a Comment

Login with GitHub to post a comment