? Success
Referenced as Pull Request for: # 7736

User tests: Successful: Unsuccessful:

avatar w00fz
w00fz
10 Mar 2014

In the previous version of Chosen, one could render a select element with class name chzn-done so that Chosen would skip it and not try to treat it with the Chosen UI.

This is no longer possible since version 3.2.3 and no alternative way to render regular select is provided.

Proposed is the addition of the check for the class chzn-done like it used to be.

JoomlaCode [#33449]

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
3.00

avatar w00fz w00fz - open - 10 Mar 2014
avatar w00fz w00fz - change - 10 Mar 2014
Labels
avatar w00fz w00fz - change - 10 Mar 2014
Labels Added: ?
avatar betweenbrain
betweenbrain - comment - 10 Mar 2014

Possible duplicate of #1620

avatar Bakual
Bakual - comment - 10 Mar 2014

I think we should not change anything in external libraries like chosen.
We really should get rid of applying chosen to each <select> and instead apply it to the (default) class advancedSelect. That would solve it without having to make changes to the library.

avatar w00fz
w00fz - comment - 11 Mar 2014

Isn't the Chosen library in J3.2.3 already modified by Joomla? I can see different <JUI> wrappers in there.
I agree though, one way or another, it should be possible to render a simple select and not force Chosen for all of them.

avatar Fedik
Fedik - comment - 12 Mar 2014

I see chzn-done in previous version, strange that they removed it,
but add data-chosen="bla" to <select> will have same effect, maybe this is cause :smile:

think this pull has the right to life :smile:
but would be good add <JUI> for part of the code that you hacked, in way as it already in chosen.js, and description on the top comment :wink:

avatar stutteringp0et
stutteringp0et - comment - 9 May 2014

There is no method to add data-chosen="bla" to config XML, while the previously available "chzn-done" class did the trick.

In the interim, I'm using javascript to destroy the chosen instance for specific inputs:

$('#elementid').chosen('destroy');

It's terribly inefficient to create the instance and then destroy it - but what's the alternative for XML?

avatar stutteringp0et
stutteringp0et - comment - 9 May 2014

Just found this commit from the chosen folks:
harvesthq/chosen#1379

They intentionally removed the chzn-done check in favor of the data attribute. So without a method to add data attributes to XML form fields, I'm stuck with a band-aid javascript solution.

avatar betweenbrain
betweenbrain - comment - 9 May 2014

It would be nice if we could get something like
#1620 merged so that extensions
can opt-in instead.

Matt Thomas
203.632.9322
http://betweenbrain.com/

Sent from mobile. Please pardon any typos or brevity.
On May 8, 2014 8:27 PM, "stutteringp0et" notifications@github.com wrote:

Just found this commit from the chosen folks:
harvesthq/chosen#1379 harvesthq/chosen#1379

They intentionally removed the chzn-done check in favor of the data
attribute. So without a method to add data attributes to XML form fields,
I'm stuck with a band-aid javascript solution.


Reply to this email directly or view it on GitHub#3274 (comment)
.

avatar stutteringp0et
stutteringp0et - comment - 9 May 2014

I can appreciate the auto-application of chosen. It keeps the interface consistent - but when we lose a feature like chzn-done and have no alternative, I get a little flustered.

I'd accept an option to add data attributes to XML fields - at least it would provide an avenue to the way "chosen" wants to handle it.

avatar brianteeman brianteeman - change - 21 Aug 2014
Status New Pending
avatar nicksavov nicksavov - change - 21 Aug 2014
Labels
avatar nicksavov nicksavov - change - 21 Aug 2014
Labels Removed: ?
avatar brianteeman brianteeman - change - 2 Sep 2014
Category JavaScript
avatar roland-d
roland-d - comment - 5 May 2015

Almost a year to the date but still valid. I have been checking this and getting data attributes into the XML forms needs some serious thinking. As @stutteringp0et already said, we already modified the chosen library, this change is not going to make it worse as it is.

Sure, we shouldn't be modifying external libraries but here it already happened. Having tested the latest version of Chozen it is unusable at the moment. So with both the new version not being usable and adding data attributes to XMLs not an easy fix, I suggest we merge as it is.

avatar JoomliC
JoomliC - comment - 21 Jul 2015

@roland-d +1 to merge this.
I'm currently working on the color picker, and since 3.2.3, the simple color picker introduced in core by @nonumber is broken (#807)
In the PR i'm working on, i have added to select a data-chosen="true" attribute which solves the issue with picket input, but until my PR will be committed, @w00fz 's PR fixes the simple color picker too! ;-)

avatar JoomliC JoomliC - test_item - 21 Jul 2015 - Tested successfully
avatar timhaeuser
timhaeuser - comment - 21 Aug 2015

Error
The patch could not be applied because the repository is missing


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

avatar timhaeuser timhaeuser - test_item - 21 Aug 2015 - Not tested
avatar roland-d roland-d - reference | 9dc2868 - 21 Aug 15
avatar roland-d roland-d - change - 21 Aug 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-08-21 13:07:52
Closed_By roland-d
avatar roland-d roland-d - close - 21 Aug 2015
avatar roland-d roland-d - close - 21 Aug 2015
avatar roland-d
roland-d - comment - 21 Aug 2015

Closing this in favour of #7736 which has an active repo.


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

Add a Comment

Login with GitHub to post a comment