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

Add shiny create option to choose from online template gallery #1277

Merged
merged 2 commits into from
Apr 2, 2024

Conversation

cpsievert
Copy link
Collaborator

This PR adds another choice shiny create to choose from the online gallery

Screenshot 2024-04-02 at 3 29 50 PM

And when chosen it opens your browser (along with a message)

@cpsievert cpsievert requested a review from schloerke April 2, 2024 20:40
shiny/_main.py Outdated
@@ -483,6 +483,7 @@ def try_import_module(module: str) -> Optional[types.ModuleType]:
"Dashboard": "dashboard",
"Multi-page app with modules": "multi-page",
"Custom JavaScript Component": "js-component",
"Choose from online gallery 🔗": "external-gallery",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe this could be "Choose from Shiny Templates gallery online"? It's hard to tell from "online gallery" that it's our online gallery. As written it kind of feels like there would be another choice to pick which online gallery.

While here I also noticed that we don't indicate which options will lead to more choices to pick the template. We should add ... where there's a submenu to pick the template, e.g. Custom JavaScript component....

Oh and we should use consistent casing and emoji on these: they should all be sentence case and if we start using emojis we should use them for all options.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks, good points.

I'm trying to keep this PR aimed at just the addition of this choice, and #1274 already addresses the casing issue, so I'll address the submenu thing there as well

Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm ok with the link emoji as it hints it's going to do something different

@cpsievert cpsievert added this pull request to the merge queue Apr 2, 2024
Merged via the queue into main with commit 8d45c26 Apr 2, 2024
32 checks passed
@cpsievert cpsievert deleted the shiny-create-gallery branch April 2, 2024 21:51
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.

None yet

3 participants