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

Inputs placeholder in Adaptive theme #5768

Open
53v3n3d4 opened this issue Dec 7, 2022 · 4 comments
Open

Inputs placeholder in Adaptive theme #5768

53v3n3d4 opened this issue Dec 7, 2022 · 4 comments

Comments

@53v3n3d4
Copy link

53v3n3d4 commented Dec 7, 2022

Description of the bug

I am trying to complete my personal st theme/color scheme. And after finishing color scheme, I notice that on my adaptive theme, the input placeholder (Find in Files / Goto Symbol in Project / Select Color Scheme...) started getting a different color. It is happening on adaptive default too. On dark and light theme it does not happen, since I am using widgets and setting foreground color (no variables>rulers defined).

Screenshot Treble Adaptive
screenshots-adaptive

Screenshot Adaptive Default
screenshots-adaptive-default

Steps to reproduce

  1. The input placeholder color on Adaptive is coming from color-scheme > variables > rulers.
  2. Set any color on color scheme like orange or cyan.
  3. Select Adaptive theme , open Find in Files or any quick panel that has placeholder like Select Color Scheme, Goto Symbol in Project...

Expected behavior

I think it should be defined in theme. Imo it is hard having some parts you define in color scheme that is related to theme. Like this placeholder or console background, popup css...

Actual behavior

The input placeholder color on Adaptive is coming from color-scheme > variables > rulers. On Dark and Light theme it does not happen.

Sublime Text build number

4143

Operating system & version

macOS 11.6.8 or 12.6

(Linux) Desktop environment and/or window manager

No response

Additional information

No response

OpenGL context information

No response

@deathaxe
Copy link
Collaborator

deathaxe commented Dec 7, 2022

Input fields are basically special kinds of views and therefore share settings and source of colors. They support syntax highlighting (see regexp, ...) and therefore colors need to be taken from color scheme.

That said, placeholders probably deserve a special global variable such as placeholders instead of using rulers.

@53v3n3d4
Copy link
Author

53v3n3d4 commented Dec 7, 2022

Hi deathaxe,

Yes I know you guys have your reasons. Just my opinion, I have to write something on bug report template (required field). I think some settings on theme would make easier for both, theme and scheme.

About placeholders, it is a good idea. Or use foreground color I think is more related than rulers. Default and Dark theme might be using it.

Thanks for reply.

@deathaxe
Copy link
Collaborator

deathaxe commented Dec 8, 2022

An alternative way to control placeholder highlighting independently would be:

A theme can specify dedicated color scheme for widgets and therefore control any highlighting aspect. The downside is those not being related with a user color scheme and thus wouldn't follow global highlighting in any way.

Widget - <Theme Name>.sublime-settings

{
	"color_scheme": "Widget - <Theme Name>.hidden-color-scheme",
}

Widget - <Theme Name>.hidden-color-scheme

{
	"name": "Widgets",
	"variables":
	{
	},
	"globals":
	{
             "rulers": "grey";
	},
	"rules":
	[
	]
}

@53v3n3d4
Copy link
Author

53v3n3d4 commented Dec 8, 2022

Hi deathaxe,

Yes, Dark and Light themes placeholders do not change because has widgets. Even if no rulers in it, only foreground it works.

{
    "name": "Widgets",
    "variables": {},
    "globals":
    {
        "background": "grey-light",
        "foreground": "grey"
    },
    "rules": []
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants