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

Custom styling and HTML #60

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

jsicherman
Copy link

Adds support for custom loaders via the addition of custom.class for specifying a CSS class name to add to the spinner, and custom.html, which allows users to specify custom HTML to provide to the spinner. Users just need to load their custom CSS as usual, pass in any necessary HTML elements and they're good to go.

@daattali
Copy link
Owner

daattali commented Oct 9, 2021

Thanks for the PR, sorry I haven't found time to dedicate to this repo in a while.

custom.class is a good idea, but what do you think about it appending to the existing class names instead of overwriting? That way if someone wants to add some of their own css, they can add the CSS, set custom.class to whatever they want, and either choose to set custom.css to be TRUE or FALSE depending on whether they usecase is to completely rewrite the look or if it's only to modify it. The documentation for custom.class should probably mention that it works in tandem with custom.css in that case.

Regarding custom.html: could you provide an example usage?

PS say hi to Paul and Ogan from me!

@daattali
Copy link
Owner

daattali commented Mar 5, 2023

@jsicherman I want to do some cleanup of old PRs - do you think you'll have time in 2023 to resolve this PR, or should we abandon it?

@jsicherman
Copy link
Author

Hey! Sorry, I totally missed this and forgot about this PR. Will have a look at this again next week to see about getting it done and dusted :)

PS: what's your connection to the PavLab? Small world!

@daattali
Copy link
Owner

daattali commented Mar 6, 2023

I did a rotation with Paul in 2013, and he also allowed me to use his lab space as my own when I did my masters with Jenny Bryan because she didn't have lab space. I always say he's one of the few real old school scientists I know :)

@daattali
Copy link
Owner

Let me know if you want to keep this PR alive

@daattali
Copy link
Owner

daattali commented May 8, 2023

This package had a lot of changes in the last 2 months. If this PR isn't getting updated, I'll close it.

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.

2 participants