Skip to content
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

patched code #156

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

patched code #156

wants to merge 12 commits into from

Conversation

kccarter76
Copy link

I have corrected my code fu pah and this is the code for this project with all my tweaks that were made to make better use of it in the jq grid widget and as a stand alone.

Added the ability to set the height either by option or for the multi select to discover the height of the container that it is rendering in and accounting the height of sibling elements.
updated the css to hide overflowing options
I have observed a bug where the 'is' method is not identifying selected options because the 'selected' attribute was not set.
Update to adjust the menu to fit the content that is to be shown. 

reason sometimes the button is 75px in width but the content is 300 pixels or in multi select mode for the header to look nice it needs to be minumum 225 pixels wide.
some widgets like jq grid offer frozen columns which basically clones the selected columns and headers from the main layer and fixes them at a relative position to give the impression of being frozen. however this code was always referencing the button of the original layer that was scrollable. as a consequence the drop down would not appear where a user would expect it to in this particular use case. This code change address's the issue by always using the button that was used to show the select list.
I have corrected the javascript and copied in my copy with the modifications made to present it for the jq grid
copy of my css for this widget
added the ability configure a means for the Ctrl changes to be canceled.
update to correct plugin for multi select
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