-
Notifications
You must be signed in to change notification settings - Fork 15
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
UI feedback on 1.2.1
#813
Comments
I think so. Maybe @sitjart can comment.
I think I agree with this, although there are some pages where you can end up deeper in a page hierarchy (managing content provider sources, and other admin things). I'll bring it up at the next TeSS club to see what others think.
Yes, probably. At the time it was not possible to get an email address under the elixir domain, however this has recently been addressed, so I'll look into replacing it with [email protected] or something.
👍 actually 1.2.1 was not officially "released" yet, but it could link to the tag. (also, confusingly, the UI redesign is actually part of the 1.3.0 release, but was was merged into 1.2.1 as part of a pre-release to show to our SAB...)
Yeah, there are some padding tweaks that need doing. Various sub elements of the page (breadcrumbs, sidebar) have their own X padding in addition to the X padding of the page wrapper, whereas the things in the navbar don't. Also we have broken/abused one of the Bootstrap layout classes ( |
* It was not applying the negative margins so any `col-` divs inside the row would not be horizontally aligned with non-row content. * It was being abused to apply vertical margins to various elements. Removed these, or replaced with helper class (e.g. `my-3`) to produce the same effect.
Also reduce padding between sort and filter in sidebar
Is your feature request related to a problem? Please describe
This is feedback on the latest release
1.2.1
as shown on the public website.Describe the solution you'd like
i
button on the Events and Materials pages should be pulled to the right of the page a bit so it aligns vertically with the rest of the elementsAdditional context
Please feel free to edit this issue and split it across several, so that it better aligns with the development work.
The text was updated successfully, but these errors were encountered: