-
Notifications
You must be signed in to change notification settings - Fork 9
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
Filter page update/fixes #67
Filter page update/fixes #67
Conversation
Fixes broken back due to form resubmit
This allows javascript to copy easily
Reduces page jumping
Tighten up fields and reduce jumping when adding terms.
Saves changes if reloaded or using back button
Just had a chance to see the chosen update. Will have to look at that some, probably shouldn't have different kinds of date pickers in different places. Would still appreciate some testing to be sure I didn't miss any filter cases in the javascript. |
chosen just does dropdowns so shouldn't interfere with date. |
Actually it's the datepicker. Never realized you'd added one to montagereview. |
Well that's just an html5 datetime input... and it is not well implemented in all browsers surprisingly. |
What's with the ui icons? We already have the ones that come with bootstrap |
2 reasons. It's part of jquery-ui and the addon was complaining about missing icons. The main reason is I didn't know we had any icons with bootstrap. I can't find them, guess I'll have to research bootstrap some. |
2b06c5b
to
a091153
Compare
d50ff72
to
6ffa96d
Compare
I changed it to use the bootstrap icons in the picker. Added chosen to most of the dropdowns, I can't get it to work well with the bracket boxes. Either they're too small and confusing due to wrapping or I have to explicitly set a width large enough for everything to fit. I'll either have to set the width inline with javascipt or just leave them as regular dropdowns. Chosen isn't really needed for those but I hate that it doesn't look consistent. |
Most drop downs don't need to be chosen. It would be nice to style the rest to look like them though. Maybe you should just steal the styling for regular drop-down
…On December 17, 2017 9:19:12 AM EST, digital-gnome ***@***.***> wrote:
I changed it to use the bootstrap icons in the picker.
Added chosen to most of the dropdowns, I can't get it to work well with
the bracket boxes. Either they're too small and confusing due to
wrapping or I have to explicitly set a width large enough for
everything to fit. I'll either have to set the width inline with
javascipt or just leave them as regular dropdowns. Chosen isn't really
needed for those but I hate that it doesn't look consistent.
--
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
#67 (comment)
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
|
Removed chosen from conjuction and brackets. Restyled dropdowns to match chosen on filters page. |
Overrides bootstrap input colors for dark theme and more chosen
Filters in javascript for a more responsive page. Also fixes form resubmit back button weirdness.
Clicking filter from anywhere should pass any appropriate filter terms including limit and sort. Limit and sort from filter page work again.
Filter page centered and compressed. Date/Time picker added. I'm not sure why jscalendar wasn't included. I used an addon for jquery-ui, MIT licensed. I believe that is all ok with Debian. We already had the jquery-ui js file, this adds the css and image files from it.
I think I tested every combination but I'm not sure. This needs some testing I think. Also limit defaults to 100 but prior to this it did not work so I'm not sure if we want that limit to apply. 1.31.1 does not have a default limit.