-
Notifications
You must be signed in to change notification settings - Fork 12
✨ Upgraded member select dropdowns to choices.js #28
Conversation
views/layout.pug
Outdated
@@ -10,6 +10,7 @@ html | |||
link(rel='stylesheet', href='/admin/css/vendors/bootstrap.css') | |||
link(rel='stylesheet', href="https://cdnjs.cloudflare.com/ajax/libs/bootstrap-social/5.0.0/bootstrap-social.min.css") | |||
link(rel='stylesheet', href="https://cdnjs.cloudflare.com/ajax/libs/bootstrap-daterangepicker/2.1.24/daterangepicker.css") | |||
link(rel='stylesheet', href='https://rawgit.com/jshjohnson/Choices/7255fa42ceb89e74f6b0b03d4868e6fec9162e39/assets/styles/css/choices.css') |
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.
Should this be local? Or at least a cdn?
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.
rawgit is a cdn, though you're right, i'm using the limited development link. will update to the production one.
views/projects/settings.pug
Outdated
//- } | ||
//- } | ||
//- $('.leads-select').multiselect(teamSelectSettings) | ||
//- $('.members-select').multiselect(teamSelectSettings) |
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.
Gonna call out the commenting of code :)
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.
word, removing.
@pltaylor this should be good for another pass. |
This is to fix the "members dropdown is way too long" issue. This adds both a truncated box and a search box for select menus.
This removed the images in the dropdown, but I think the added functionality outweighs that minor piece of sugar.
screenshot:
cc @pltaylor