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

Issues with light theme #92

Open
Ranguna opened this issue Aug 14, 2022 · 11 comments
Open

Issues with light theme #92

Ranguna opened this issue Aug 14, 2022 · 11 comments

Comments

@Ranguna
Copy link

Ranguna commented Aug 14, 2022

When using pywal to generate a light theme and applying it in firefox, some text becomes unreadable:
The active tab:
image

The extensions itself:
image

The reason I want to explicitly use the light option though wal cli is because some tools like the terminal and vscode only change to the light option when generating the colors with that option.

Changing to the light option on the extensions makes things a bit more unreadable:
image

Is there any way to make this extensions compatible with colors generated with the light cli option ?

@Frewacom
Copy link
Owner

Frewacom commented Aug 14, 2022

You can modify the template used for picking the colors, but that is not really a solution. I don't quite remember how the .cache/wal/colors.json file look - does it include anything that would indicate that it is a "light" theme? If not, I guess pywalfox could check the brightness of some colors to try and figure out whether it is a light or dark theme.

@Ranguna
Copy link
Author

Ranguna commented Aug 14, 2022

Wow thanks for the fast reaction to this one!

As for whether colors.json indicates which color scheme is in use, unfortunately it does not :(
Here's my colors.json at the moment:

{
    "wallpaper": "/path/to/my/image.png",
    "alpha": "100",

    "special": {
        "background": "#f9f9f9",
        "foreground": "#3e3e3e",
        "cursor": "#3e3e3e"
    },
    "colors": {
        "color0": "#f9f9f9",
        "color1": "#6e535d",
        "color2": "#93949f",
        "color3": "#bfa2ab",
        "color4": "#a8bcc8",
        "color5": "#ccabc3",
        "color6": "#cdbcb3",
        "color7": "#3e3e3e",
        "color8": "#bababa",
        "color9": "#6e535d",
        "color10": "#93949f",
        "color11": "#bfa2ab",
        "color12": "#a8bcc8",
        "color13": "#ccabc3",
        "color14": "#cdbcb3",
        "color15": "#3e3e3e"
    }
}

I also created dylanaraps/pywal#685, with the hopes that maybe we could standardize this somehow, either by including both color schemes in colors.json or not recommending consumers to have a dark/light option in each application.

If not, I guess pywalfox could check the brightness of some colors to try and figure out whether it is a light or dark theme.

Is there a reason you need to know which theme is active ?
The vscode extension and the terminal emulator theme simply uses the active theme and won't give you the option to toggle; if you want to change, you'll have to run the wal command with or without the -l flag.
So maybe the same could be done in pywalfox (if wal does not provide two color scheme in the colors.json): not having the option to choose between dark and light and simply use the colors in colors.json.

@Frewacom
Copy link
Owner

Frewacom commented Aug 14, 2022

Is there a reason you need to know which theme is active ?

I think the colors that should be used when theming the browser is different for light and dark mode. Using the same color indicies for light mode results in a very ugly looking theme, which is why I do some manual adjustments to the colors in the code when light mode is activated. I haven't looked at this in a long time, so I might be wrong though.

@Ranguna
Copy link
Author

Ranguna commented Aug 14, 2022

Makes sense, I can try taking a look at this in the future and check which colors the terminal and vscode use for text and focus text, thanks for all the info!

@holmescharles
Copy link

Any progress on this? I am experiencing the same issue.

@Packjackisback
Copy link

Yep, same thing, the only way I see now is forking it.

@Ranguna
Copy link
Author

Ranguna commented Apr 11, 2024

@Packjackisback or propose a PR

@holmescharles
Copy link

@Ranguna, this repo does not seem to be maintained anymore, unfortunately. So a fork would be best.

@Ranguna
Copy link
Author

Ranguna commented Apr 11, 2024

Hmmm maybe
@Frewacom are you still around?

@Frewacom
Copy link
Owner

@Frewacom are you still around?

Yes, I am just busy with other stuff. PRs are appreciated.

@Ranguna
Copy link
Author

Ranguna commented Apr 14, 2024

Yes, I am just busy with other stuff. PRs are appreciated.

Good to know, thanks!

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

No branches or pull requests

4 participants