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

modernize select multiple (use jquery chosen) #12

Merged
merged 4 commits into from Oct 24, 2014

Conversation

glensc
Copy link
Member

@glensc glensc commented Oct 22, 2014

works in "composer" mode:

$ composer update

@@ -22,8 +22,8 @@
</tr>
<tr>
<td>
<select name="users[]" size="5" multiple>
{html_options options=$users}
<select name="users[]" size="5" multiple style="width:180px;" data-placeholder="{t}Choose Assignees...{/t}" class="chosen-select">
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we move this to inline css into page.css? I see it quite a few other places as well.

Copy link
Member Author

Choose a reason for hiding this comment

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

hmm post_note.php page lacks <body> css classes:

</head>
<!--[if lt IE 7 ]> <body class="ie6  " id=""> <![endif]-->
<!--[if IE 7 ]> <body class="ie7  " id=""> <![endif]-->
<!--[if IE 8 ]> <body class="ie8  " id=""> <![endif]-->
<!--[if !IE]>--> <body class=" " id=""><!--<![endif]-->
<div id="container">

balsdorf pushed a commit that referenced this pull request Oct 24, 2014
@balsdorf balsdorf merged commit 266a3af into eventum:master Oct 24, 2014
@glensc glensc deleted the jquery-chosen branch October 25, 2014 19:29
@glensc glensc modified the milestone: 3.0.0 Feb 10, 2015
@glensc glensc changed the title modernize select multiple modernize select multiple (use jquery chosen) Jul 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

2 participants