-
Notifications
You must be signed in to change notification settings - Fork 6
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
Voila mode #340
Voila mode #340
Conversation
This reverts commit 89146b1.
766888c
to
71c45e4
Compare
This should not be merged until Voila support lands in an |
@@ -1033,6 +1033,11 @@ define([ | |||
$subdirsBox.prop('checked', false); | |||
} | |||
$subdirsBox.parent().toggleClass('rsc-text-light', !canIncludeSubdirs); | |||
|
|||
// Disable hide code cells options in Voila mode, since Voila always hides them. |
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.
The code cell boxes are already greyed out when Voila mode is selected. Is there more that should be done to complete #348 ?
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.
We should review the docs on hiding code cells to see if any updates are needed.
Description
This PR implements a 3rd mode button to publish in Voila mode.
Connected to #https://github.com/rstudio/connect/issues/22745