? Pending

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
9 May 2017

Pull Request for Issue # .

Summary of Changes

  • Introduce Choices.js: https://github.com/jshjohnson/Choices
  • Replace chosen in the the category on the fly field
  • Chosen will still be shipped (aim is that core will not use it in J4)
  • Needs styling to match the templates

Testing Instructions

Edit an article and try to insert a category on the fly:
screen shot 2017-05-09 at 18 24 31

ATM there is a PHP error for Table Category not supported. File not found. So @laoneo and @joomdonation I think is related to namespacing...

Expected result

Actual result

Documentation Changes Required

UPD. This is on hold till we send a PR upstream for: jshjohnson/Choices#163

avatar dgt41 dgt41 - open - 9 May 2017
avatar dgt41 dgt41 - change - 9 May 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 May 2017
Category JavaScript Repository Administration com_categories com_content
avatar Bakual
Bakual - comment - 9 May 2017

Chosen will still be shipped (aim is that core will not use it in J4)

What about deprecate in J3.8 and remove chosen with J4 but add choices into J3.8 (or 3.9) so extensions could already use it? Imho there isn't much point dragging chosen along the whole J4.
Or is there a reason you want to keep it?

avatar dgt41
dgt41 - comment - 9 May 2017

@Bakual choices are not IE8 compatible...

dbe7c4d 9 May 2017 avatar dgt41 CS
avatar dgt41 dgt41 - change - 9 May 2017
Labels Added: ?
avatar joomdonation
joomdonation - comment - 9 May 2017

@dgt41 That's a problem with the choice of creating model object directly from the class. I made a temp fix dgt41#53 for now. The backend team needs to find a proper solution for it later

43a9231 9 May 2017 avatar dgt41 oops
avatar dgt41
dgt41 - comment - 9 May 2017

@joomdonation it works nicely now, thanks!

avatar Bakual
Bakual - comment - 9 May 2017

choices are not IE8 compatible...

@dgt41 So what? As long as core doesn't use it, it's no problem to include it in J3.8/9. We're not breaking B/C here. As for 3rd party extensions using it, it's up to them if they want to support IE8 or not. They don't have to support the same browsers as core does.

avatar C-Lodder
C-Lodder - comment - 10 May 2017

@dgt41 - Oh forgot to say, have you deprecated chosen in 3.x?

avatar ciar4n
ciar4n - comment - 12 May 2017

I have tested this item successfully on bd6779f


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

avatar ciar4n ciar4n - test_item - 12 May 2017 - Tested successfully
avatar brianteeman brianteeman - change - 8 Jun 2017
Milestone Added:
avatar brianteeman brianteeman - change - 8 Jun 2017
Milestone Added:
avatar dgt41 dgt41 - change - 22 Jul 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-07-22 20:03:57
Closed_By dgt41
avatar dgt41 dgt41 - close - 22 Jul 2017
avatar dgt41 dgt41 - change - 24 Jul 2017
Status Closed Pending
Closed_Date 2017-07-22 20:03:57
Closed_By dgt41
avatar dgt41 dgt41 - reopen - 24 Jul 2017
avatar dgt41 dgt41 - change - 24 Jul 2017
Title
[4.0] Replace chosen with choices js
[4.0][On Hold] Replace chosen with choices js
avatar dgt41 dgt41 - edited - 24 Jul 2017
avatar dgt41 dgt41 - change - 24 Jul 2017
The description was changed
avatar dgt41 dgt41 - edited - 24 Jul 2017
avatar dgt41
dgt41 - comment - 28 Jan 2018

Closing here, we will try to create a custom element tag that will satisfy all our needs (accessibility, drag and drop, etc)

avatar dgt41 dgt41 - change - 28 Jan 2018
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2018-01-28 11:33:01
Closed_By dgt41
avatar dgt41 dgt41 - close - 28 Jan 2018

Add a Comment

Login with GitHub to post a comment