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

--update-interactive output does not support light theme in terminal #7439

Closed
2 of 4 tasks
ai opened this issue Dec 18, 2023 · 4 comments · Fixed by #7454
Closed
2 of 4 tasks

--update-interactive output does not support light theme in terminal #7439

ai opened this issue Dec 18, 2023 · 4 comments · Fixed by #7454

Comments

@ai
Copy link
Contributor

ai commented Dec 18, 2023

Verify latest release

  • I verified that the issue exists in the latest pnpm release

pnpm version

8.12.1

Which area(s) of pnpm are affected? (leave empty if unsure)

CLI

Link to the code that reproduces this issue or a replay of the bug

No response

Reproduction steps

  1. Enable light theme in terminal
  2. Run pnpm update-interactive

Describe the Bug

Selection checkboxes are invisible

Screenshot from 2023-12-18 12-26-54

Screenshot from 2023-12-18 12-26-37

Expected Behavior

Use gray color for checkbox instead of #fff. Named colors are safe for different color themes.

Which Node.js version are you using?

20.10.0

Which operating systems have you used?

  • macOS
  • Windows
  • Linux

If your OS is a Linux based, which one it is? (Include the version if relevant)

Fedora 38

@ai ai added the type: bug label Dec 18, 2023
@KSXGitHub
Copy link
Contributor

You can try tweak it in pkg-manager/plugin-commands-installation/src/update/index.ts and create a PR.

@ai
Copy link
Contributor Author

ai commented Dec 23, 2023

@KSXGitHub thanks for pointing the file.

PR is here #7454

@zkochan
Copy link
Member

zkochan commented Dec 23, 2023

In what terminal is the issue reproducible? In alacritty (on macOS), I see the selection boxes (I use a light theme).

@ai
Copy link
Contributor Author

ai commented Dec 23, 2023

Black Box, Linux. The visibility depends on theme, but not great in most of them.

Screenshot from 2023-12-23 20-28-50
Screenshot from 2023-12-23 20-28-40
Screenshot from 2023-12-23 20-28-44

Here is a standard GNOME Terminal with default theme (visible, but hard to read)
Screenshot from 2023-12-23 20-31-05

I agree that white doesn’t work completely only on some themes. But in general, I think it is not a “safe” color.

zkochan pushed a commit that referenced this issue Dec 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants