Dashboards: multiple enhancements #598
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Yearly costs
- Add an option to display all the data.
- Add translation
- Update template pot file
…ds' dropdown lists
…nd' into wip_stockholm
…' into wip_stockholm
…nto wip_stockholm
…ds' dropdown lists - Fix limits for export - Fix export parameters summary
…ds' dropdown lists - Fix select names and ids to be both compatible with xhr and post
…ds' dropdown lists - Fix cost details for yearly dashboard
- Add fixed headers display
- Add striping and hovering
- Add classes for styling totals and subtotals
- Add cell border - Fix striping and hovering
- Fix total and subtotal classes - Add styling to total and subtotal rows
- Add underline to filtering headers
…ds' dropdown lists - Fix order type filtering
- Fix order type filtering
…ds' dropdown lists - Add reset buttons to dashboards options forms
- Move search field to the left (more easily accessible) - Add some space between the title and the content
|
Tested this PR extensively and the new enhancements works as intended. But as Stockholm University Library has sponsored these developments, it is probably good to have someone else test it as well. |
|
As approved by the SC, I'm merging this. |
This was referenced Nov 21, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR is a superset of the following PRs:
If the changes provided by all previous PRs are accepted, I'd rather see this branch being merged, avoiding inter-PRs merge issues.
Before:
After:
