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

Please make the download options configurable. #2918

Open
maxtheman opened this issue Nov 21, 2024 · 4 comments
Open

Please make the download options configurable. #2918

maxtheman opened this issue Nov 21, 2024 · 4 comments
Labels
enhancement New feature or request

Comments

@maxtheman
Copy link
Contributor

Description

I'm hoping to be able to make it so I can disable different download options for the notebook. In particular, I would like to remove "download as HTML" to prevent confusion with my users.

Image

Suggested solution

Ideally, it's configurable somewhere in the app, maybe even you just put in tools.marimo when that comes out in the pyproject.

Alternative

Here's some CSS that removes it for now. You could customize this to remove the other option based on the text or the icon.

[role="menuitem"]:has(:contains("Download as HTML")) {
    display: none !important;
}
[role="menuitem"]:has(.lucide.lucide-folder-down) {
    display: none !important;
}

Result:Image

Additional context

No response

@maxtheman maxtheman added the enhancement New feature or request label Nov 21, 2024
@mscolnick
Copy link
Contributor

thanks for sharing your workaround, we can add this. (i may consider using your workaround as the solution and programmatically add/remove css 😄 )

@diehl
Copy link

diehl commented Nov 21, 2024

Is there a way via CSS to remove the entire menu?

@maxtheman
Copy link
Contributor Author

@mscolnick you're welcome to it!

@diehl here's one that would remove the whole menu. I'm just looking at the icon and selecting for that, with specificity on the button, so you should be able to copy that for any item you want to hide really.

button[aria-haspopup="menu"]:has(.lucide-ellipsis) {
    display: none !important;
}

mscolnick added a commit that referenced this issue Nov 21, 2024
@diehl
Copy link

diehl commented Nov 21, 2024

Thank you @maxtheman! My HTML/CSS skills are basically nonexistent so this a significant help ;-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants