Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Right to Left initialization option #1510

Merged
merged 6 commits into from
Sep 19, 2016
Merged

Right to Left initialization option #1510

merged 6 commits into from
Sep 19, 2016

Conversation

kenearley
Copy link

@harvesthq/chosen-developers I've pulled in @koenpunt's branch and updated the docs to note deprecation of the classname approach.

@tjschuck
Copy link
Member

Made a couple of small tweaks to the docs and grammar.

@kenearley
Copy link
Author

Thanks @tjschuck

@stof
Copy link
Collaborator

stof commented Aug 26, 2013

👍

@@ -18,6 +18,7 @@ class AbstractChosen
@mouse_on_container = false
@results_showing = false
@result_highlighted = null
@is_rtl = @options.rtl || /\bchosen-rtl\b/.test(@form_field.className) || false
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.test returns true or false so there's no need for the final || false.

@kenearley
Copy link
Author

Is this still relevant? We should just close this if it's stale.

@koenpunt
Copy link
Collaborator

Why would it be irrelevant?

@koenpunt koenpunt merged commit 2ea2b46 into master Sep 19, 2016
@koenpunt
Copy link
Collaborator

Why did this linger for so long?

@koenpunt koenpunt deleted the koenpunt-rtl-option branch September 19, 2016 16:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants