-
Notifications
You must be signed in to change notification settings - Fork 198
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
picker text color changes to black in dark theme on mouse hover #1852
Labels
bug
Results from a bug in the CSS implementation
Comments
@wesjohns
|
|
also, there seems to be no other places with such issues other than this :
|
pfulton
added a commit
that referenced
this issue
May 10, 2023
6 tasks
pfulton
added a commit
that referenced
this issue
May 10, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Steps to reproduce
1.https://dev-main.illustrator.adobe.com/
2. Change color theme to dark by going to file menu and settings screen
3. now do mouse hover on color theme picker . it will be white initially before hover but on hover it changes to black
I see that ' #button:hover color ' is causing this issue of text color change on hover in picker.
#button:hover {
background-color: var( --highcontrast-picker-background-color-default,var( --mod-picker-background-color-hover,var(--spectrum-picker-background-color-hover) ) );
border-color: var( --highcontrast-picker-border-color-default,var( --mod-picker-border-color-hover,var(--spectrum-picker-border-color-hover) ) );
color: var( --highcontrast-picker-font-color-default,var(--mod-picker-font-color-hover,(--spectrum-picker-font-color-hover)) );
}
when i uncheck #button:hover color in devtools, i dont see issue.
to do this unchecking programmatically (as hack fix )
i tried below code but it does not work :
pickerElement.setAccessibilityIdentifier(pickerId);
console.log("pdhotreprint setAccessibilityIdForColorTheme()");
//document.head.insertAdjacentHTML('beforeend', '<style>#button:hover{color:'';}</style>');
Expected behavior
Screenshots
Environment
Additional context
The text was updated successfully, but these errors were encountered: