-
Notifications
You must be signed in to change notification settings - Fork 2
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
Improve UI #87
Improve UI #87
Conversation
…st header UI, and gives undesireable result in some resolutions)
1356d5b
to
1d43648
Compare
title = tags$a( | ||
href = "/", | ||
tags$img(src='www/gcp_logo.png', height = '40', width ='180') |
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.
For us, this small change is quite useful since it gives us a possibility to navigate back to the home page within ShinyProxy. In later updates, we can make this more flexible so that the user can set the link and/or the logo.
@jthompson-arcus and @aclark02-arcus I will merge this PR due to time restrictions and since it is just a UI update (although I personally quite like these changes!). If you encounter any problems with it, please let me know in a follow-up issue report. |
Improvements:

Better use of space, grouping of buttons, slightly tweak facet height:
Furthermore, a bit better text sizing of value box texts by setting a minimum text size:

And the figure legend is now bigger but hidden in a popover so that it does not distract from the main content:
