Skip to content

Conversation

@copypastedeveloper
Copy link

i've merged in your latest change, so this should be definitive. i made a few changes to allow for something i needed, as well as setting up what i think is a better way to configure the options. also added a couple of features i thought were there but were not.

ncgonz and others added 7 commits March 15, 2011 22:02
…h of the dropdown, fleshed out topOffset and leftOffset, reworked the way the configuration is merged together to better handle defaults.
…select list to be smaller than the styled elements.
Conflicts:
	jquery.yaselect.js
	jquery.yaselect.min.js
@choonkeat
Copy link
Owner

Hi ncgonz,

Thanks for your patch! There's a few things I need your help with though,

  • could you help restore the source code to "2-spaces soft-tab" indent instead of "8-spaces hard-tab"?
  • anchor is kind of perfunctory (for position:relative only) and shouldn't be styled. can useSelectCss options be applied to wrap instead of anchor?
  • the minimized file was generated by closure compiler. If you could use the same, that would be great. Otherwise I'll be overwriting it in a separate commit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants