New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#84 #86 #88 update to newer Chosen lib #97
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice work!
@@ -53,7 +53,7 @@ function redirectIfQuery() { | |||
var pageName = window.location.toString(), | |||
delimiterPosition, | |||
redirectURL = 'https://github.com/Esri?q=', | |||
topicList = document.getElementById('esri-topic-list').childNodes | |||
topicList = document.getElementById('esri-topic-list').childNodes, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this why we were getting redirects on urls with no search string?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it seemed to me that we were redirecting with no query even if we received no query, so I changed it to only check if a query was requested
@@ -1,10 +1,1321 @@ | |||
// Copyright (c) 2011 Harvest http://getharvest.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we could make this PR even redder if we just used a cdn
load chosen plugin from cdn
Let's try this out and see if this is a better UI. This is the latest version of Chosen, but it's a bit different than what we had. May require more CSS work.