-
Notifications
You must be signed in to change notification settings - Fork 454
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
Include a dark theme in the default image? #347
Comments
Yes please, the glare from the phpmyadmin page late at night is a bit much :) |
Oh man, this would be great! <3 |
+1 |
2 similar comments
+1 |
+1 |
We have this in mind, it requires some team discussion but for me it's okay to add one more theme by default (Only for Docker users) |
+1 on this. Just installed phpMyAdmin for the very first time (using Docker) and the first thing I looked for was how to set dark theme. The older I get, the more light themes hurt my eyes. |
I created a custom docker image which is automatically built on top of the official phpmyadmin fpm-alpine. Docker hub: https://hub.docker.com/r/beeyev/phpmyadmin-lightweight |
@beeyev That’s awesome! I wonder what the accessibility might be of turning this into a proper PR for the core team. Maybe then it’d just get rolled into a version release? |
@ibennetch @MauricioFauth what do you think between:
|
@williamdes Is there a way to add a custom theme to an image? Doesn't makes much sense to me to ship a theme only on Docker. And I don't like the idea of ship a new theme with 5.2. |
Yes, mounting it directly in the themes folder.
yeah, that feels weird
I agree So one of the alternatives I was thinking of would be to provide some ENV to install themes from our website on container start. Or copy them from a folder (that that's not the best) |
Makes sense to build an extra image with more themes or an new image with only the theme, that can be attached to the main image? (I don't have much experience with Docker) |
It has no sense like that but if you convert "image" with "tag" then it could make sense, yes: https://hub.docker.com/_/phpmyadmin/tags But with an ENV, any existing variant can benefit from the change without much effort. Or we just add more themes by default to the Docker image on all variants but like you said, it's not best too |
So, the best thing right now is to add the ability to add more themes via an ENV. About a dark theme, I'm working on adding support (phpMyAdmin 6.0) for a button that toggles a light/dark mode for supported themes, starting with the, already included, Bootstrap theme. |
I'd like to just add a dark theme to the full regular official release, which would of course also be included in the Docker image. We have three good candidates for that right now, but darkwolf has been around for quite a while. As an aside, I'd like to make it easier for people to upgrade themes between versions, I feel right now it's mostly trial-and-error and testing. I had hoped that https://github.com/phpmyadmin/phpmyadmin/wiki/Theme_Changes would help but I'm not sure if that's really that useful or being updated. I would say that using an ENV tag to specify a theme to download would be acceptable, but my personal favorite solution is to allow users to mount the theme directory and manually put them there. I do not think we should create more Docker tags. |
I don't like the idea of adding a theme in 5.2, but I don't opposed to that, as there are no dark themes in it. And the dark themes on the demo server are good ones. |
Love the dialog and progression, it feels late to be making a solution myself. What can we do to get this going? I'm not sure why wait for a major version. I think a progression of: |
Okay, let's make a PR for this. It would be super helpful |
Actually it's quite easy to install any dark theme manually and put additional config into the docker compose.
I see there are already some preinstalled themes and it will be REALLY great to have several dark options. I use almost all in dark mode so far just to keep my eyes in a good share expecially when you code at night. And I think that most of the users of PMA are coders and use probably VSCode and there is the dark theme installed by default, so better to keep the modern pace and integrate dark themes, @MauricioFauth |
I looked back and read through the issue re: adding a more modern theme.
I’m wondering now if it might be widely accepted as a good idea to include a dark theme—even a lightweight one—as part of the default image.
Happy to be shut down. Just thinking many might like it. I’m not even picky about which theme, but just to throw something out there it looks like DarkWolf might be ok.
Thanks in advance for even considering it!
The text was updated successfully, but these errors were encountered: