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

navbar and row changes #104

Merged
merged 1 commit into from
Dec 8, 2024
Merged

navbar and row changes #104

merged 1 commit into from
Dec 8, 2024

Conversation

alan-y
Copy link
Collaborator

@alan-y alan-y commented Nov 22, 2024

Made some changes to the Shiny CSS file so that the navigation bar covers the whole width of the page and so the heading text in tabpanels lines up with the text in fluid rows. It looks all fine to me when I load up the shiny template but let me know if you think these changes may cause issues thanks.

@Moohan
Copy link
Member

Moohan commented Nov 25, 2024

I haven't checked the changes yet, but just a note to say we should make sure to update inst/text/phs_style.css inline with this one.

@alan-y
Copy link
Collaborator Author

alan-y commented Nov 25, 2024

I haven't checked the changes yet, but just a note to say we should make sure to update inst/text/phs_style.css inline with this one.

phs_style.css is for html, etc while these changes mainly affect Shiny. The user has the option to add the Shiny CSS stuff into phs_style.css when they run the function.

Copy link
Member

@Moohan Moohan left a comment

Choose a reason for hiding this comment

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

Looks good to me. Subtle changes but definitely an improvement!

Note I also just checked these using the template - It would be good if someone who has developed a dashboard from the template could try the new CSS with that.

Copy link
Collaborator

@rubenv95 rubenv95 left a comment

Choose a reason for hiding this comment

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

Also tested in the template, all fine, but I don't have a full app to test on currently.

I noticed the template is trying to source a script that doesn't exist and was throwing a warning, but that might be a separate issue? The contact page server script doesn't exist in pages/ folder.

remotes::install_github("Public-Health-Scotland/phstemplates", ref = "feature/css-changes")
phstemplates::phsshinyapp("Test/", app_name = "TestApp")
shiny::runApp('Test')
Listening on http://127.0.0.1:5610
Warning in file(filename, "r", encoding = encoding) :
cannot open file 'pages/contact_page.R': No such file or directory
Warning: Error in file: cannot open the connection
40: file
39: source
38: server [/mnt/homes/rubenv01/Test/app.R#86]
1: shiny::runApp
Error in file(filename, "r", encoding = encoding) :
cannot open the connection

@alan-y
Copy link
Collaborator Author

alan-y commented Nov 26, 2024

Also tested in the template, all fine, but I don't have a full app to test on currently.

I noticed the template is trying to source a script that doesn't exist and was throwing a warning, but that might be a separate issue? The contact page server script doesn't exist in pages/ folder.

remotes::install_github("Public-Health-Scotland/phstemplates", ref = "feature/css-changes")
phstemplates::phsshinyapp("Test/", app_name = "TestApp")
shiny::runApp('Test')
Listening on http://127.0.0.1:5610
Warning in file(filename, "r", encoding = encoding) :
cannot open file 'pages/contact_page.R': No such file or directory
Warning: Error in file: cannot open the connection
40: file
39: source
38: server [/mnt/homes/rubenv01/Test/app.R#86]
1: shiny::runApp
Error in file(filename, "r", encoding = encoding) :
cannot open the connection

Hi @rubenv95, thanks for testing the template. I just ran the install command above and the app seems to be running fine for me with the contact page on it (I added this as a suggestion on the Shiny template). It seems to me that a lot of people have customised their CSS file for Shiny apps - so I imagine the one we have may not work perfectly for quite a few apps...

@alan-y alan-y merged commit e1f0fc9 into main Dec 8, 2024
8 checks passed
@alan-y alan-y deleted the feature/css-changes branch December 8, 2024 16:39
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.

3 participants