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

feat: start work on lightswitch #5

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

feat: start work on lightswitch #5

wants to merge 1 commit into from

Conversation

maelle
Copy link
Contributor

@maelle maelle commented Oct 7, 2024

Themes don't work with the lightswitch (r-lib/pkgdown#2679 + I tried it here and the light theme looked awful).

From the Bootswatch flatly theme, @krlmlr what would you say defines the R-DBI "visual identity"? I know we kept https://r-dbi.org/ pretty close to the defaults.

@maelle
Copy link
Contributor Author

maelle commented Oct 7, 2024

In any case my idea is to fix r-dbi/RSQLite#522 by adding support for pkgdown's dark mode instead of relying on users' machines, if that makes sense.

@krlmlr
Copy link
Member

krlmlr commented Oct 7, 2024

Thanks. Back then, I picked flatly because it looked nicest to me. I can't contribute much here, sorry. We could get support from a specialist if needed.

@maelle
Copy link
Contributor Author

maelle commented Oct 11, 2024

would you be happy to just go with the default pkgdown theme or do you want to customize anything right now? If we go with the default theme, we can add the light switch and solve that user's issue.

@krlmlr
Copy link
Member

krlmlr commented Oct 11, 2024

I don't follow, flatly has a light switch: https://bootswatch.com/flatly/ .

@maelle
Copy link
Contributor Author

maelle commented Oct 11, 2024

The thing is, the light switch from pkgdown does not work with themes (yet?). "(Themes are unlikely to work with the light switch, but you can try it and see.)" in https://pkgdown.r-lib.org/articles/customise.html#bootswatch-themes (I did try and did see 😁 )

@maelle
Copy link
Contributor Author

maelle commented Oct 11, 2024

actually maybe it's undocumented but a bit better than I thought, let me try something.

@maelle
Copy link
Contributor Author

maelle commented Oct 11, 2024

ah no but I'll open an issue.

@maelle
Copy link
Contributor Author

maelle commented Oct 11, 2024

r-lib/pkgdown#2799

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