?
avatar brianteeman
brianteeman
23 Jan 2020

I made a PR #25335 that makes sure that we are using the joomla.form.field.list-fancy-select layout when we have a multiple select list field.

@wilsonge closed it in the mistaken belief that you can do multiselects natively in the browser. Well you can do them but they look like hell as shown below.

Before

image

After

image

avatar brianteeman brianteeman - open - 23 Jan 2020
avatar joomla-cms-bot joomla-cms-bot - change - 23 Jan 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 23 Jan 2020
avatar jwaisner
jwaisner - comment - 23 Jan 2020

I agree @brianteeman. It looks much cleaner in the proposed change.


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

avatar richard67 richard67 - change - 27 Jan 2020
Status New Confirmed
avatar brianteeman
brianteeman - comment - 31 Jan 2020

I would re-open the PR but it looks like because it was closed by a maintainer I cannot - @wilsonge please!!

avatar wilsonge
wilsonge - comment - 6 Feb 2020

I can't because it says it's been force pushed/recreated since it was closed

avatar wilsonge
wilsonge - comment - 6 Feb 2020

Tbh I'm really reluctant to reintroduce this everywhere. I want it still class based please. Not global select applying like we made the mistake with chosen

avatar brianteeman
brianteeman - comment - 6 Feb 2020

ok I will make a new PR

Can you still check that PR as I believe the approach there is correct

avatar wilsonge
wilsonge - comment - 6 Feb 2020

Ahh yes should be fine. sorry i should have re-reviewed before commenting rubbish

avatar brianteeman
brianteeman - comment - 6 Feb 2020

yes you should ;)

avatar brianteeman
brianteeman - comment - 6 Feb 2020

Closed see #27833

avatar brianteeman brianteeman - close - 6 Feb 2020
avatar brianteeman brianteeman - change - 6 Feb 2020
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2020-02-06 23:43:55
Closed_By brianteeman

Add a Comment

Login with GitHub to post a comment