Skip to content

Selectmenu should update its dropdown height on content update #133

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

Closed
trollixx opened this issue Jun 9, 2011 · 7 comments
Closed

Selectmenu should update its dropdown height on content update #133

trollixx opened this issue Jun 9, 2011 · 7 comments

Comments

@trollixx
Copy link

trollixx commented Jun 9, 2011

For example, deleting some items from selectmenu doesn't force recalculation of its height. Calling $("#el").selectmenu() doesn't do the thing too.

@fnagel
Copy link
Owner

fnagel commented Jun 14, 2011

Im not able to reproduce your issue. In which browsers does this problem occur? Are you able to provide a demo? Please see here: #61

@trollixx
Copy link
Author

Just now got this bug in:

  • Chromium 13.0.782.20
  • Firefox 4 & 5
  • Opera 11.11

Playing with jsfiddle to provide a demo....

@fnagel
Copy link
Owner

fnagel commented Jun 14, 2011

Ive tested in FF4 & Chrome too, without problems. I'll waiting for the demo
fiddle :-)

On Tue, Jun 14, 2011 at 22:50, trollixx <
reply@reply.github.com>wrote:

Just now got this bug in:

  • Chromium 13.0.782.20
  • Firefox 4 & 5
  • Opera 11.11

Playing with jsfiddle to provide a demo....

Reply to this email directly or view it on GitHub:
#133 (comment)

@trollixx
Copy link
Author

Got It! http://jsfiddle.net/trollixx/qj2YK/
The main condition is vertical scrollbar in selectmenu's dropdown.

@fnagel
Copy link
Owner

fnagel commented Jun 16, 2011

Issue confirmed, no idea why I did not notice this one :-( Thanks a lot!

This only occurs if a auto maxHeigth is set. This happens if there are many options before refreshing to less options.

Should be fixed as I improved the whole height & maxHeight algorithm. Please test latest version and give feedback!

Related to: #26

@trollixx
Copy link
Author

Cool, everything works like it should! Thanks for fix)

@fnagel
Copy link
Owner

fnagel commented Jun 16, 2011

Closed, please reopen if needed.

@fnagel fnagel closed this as completed Jun 16, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants