Skip to content
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

changed font size of app and changed size of sidebar #188

Merged
merged 7 commits into from
Jan 8, 2025

Conversation

anna-abelman
Copy link
Collaborator

@Paul-Carvalho can you see if this looks better on your laptop? It looks small on mine, but not sure if that is the case for other laptops

@Paul-Carvalho
Copy link
Collaborator

@anna-abelman, looks a little better but still a bit too crowded when viewing on my laptop - I attached a screenshot of what the gui upload page looks like on my laptop. Also, the text is pretty small now when viewing from my other monitor. Could we set the text size to a relative value that will scale with screen size? I wonder if we could do the same with the sidepanel, because I think the large sidepanel size when viewing from a laptop screen crowds the main panel.

fishset_gui

@Paul-Carvalho
Copy link
Collaborator

Resolves #186

@Paul-Carvalho Paul-Carvalho linked an issue Dec 30, 2024 that may be closed by this pull request
…ze. made sidebar width smaller. fixed weird action button formatting.
@anna-abelman
Copy link
Collaborator Author

@Paul-Carvalho let me know if this looks better. Also, this change made the position of the new notifications awkward, but I wasn't sure how to fix it. Do you mind taking at look at this?

Copy link
Collaborator

@Paul-Carvalho Paul-Carvalho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@anna-abelman, let me know if you have any questions about comments I added. Overall the small font scale looked good on my laptop and external monitor. The big theme looked a bit crowded on both monitors for me.

inst/ShinyFiles/MainApp/fleet_helpers.R Outdated Show resolved Hide resolved
inst/ShinyFiles/MainApp/fleet_helpers.R Outdated Show resolved Hide resolved
DESCRIPTION Outdated Show resolved Hide resolved
inst/ShinyFiles/MainApp/server.R Outdated Show resolved Hide resolved
inst/ShinyFiles/MainApp/ui.R Outdated Show resolved Hide resolved
inst/ShinyFiles/MainApp/ui.R Outdated Show resolved Hide resolved
@anna-abelman anna-abelman merged commit 4bf4768 into master Jan 8, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update GUI appearance
2 participants