Skip to content

fix(input-time-picker): invert text color on Windows when each masked input is focused #12130

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

Merged

Conversation

eriklharper
Copy link
Contributor

@eriklharper eriklharper commented May 12, 2025

Related Issue: #12122

Summary

This inverts the text color of each focused input in calcite-input-time-picker for Windows browsers whose system highlight color doesn't provide sufficient contrast by setting the text color to use the system HighlightText keyword.

For more information: https://css-tricks.com/system-things/

Tested on Chrome, Firefox and Edge on Windows 10 Enterprise build 19045.5737.

Chrome Version 136.0.7103.93 (Official Build) (64-bit)
image

Firefox Version 138.0.1 (64-bit)
image

Edge Version 136.0.3240.64 (Official build) (64-bit)
image

@eriklharper eriklharper requested a review from jcfranco May 12, 2025 22:43
@github-actions github-actions bot added the bug Bug reports for broken functionality. Issues should include a reproduction of the bug. label May 12, 2025
Copy link
Member

@geospatialem geospatialem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lookin' good on Windows! ✨🏆✨

image

@eriklharper eriklharper requested a review from driskull May 12, 2025 23:40
Copy link
Member

@driskull driskull left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Contributor

@ashetland ashetland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Noice

@eriklharper eriklharper added the pr ready for visual snapshots Adding this label will run visual snapshot testing. label May 12, 2025
@eriklharper eriklharper merged commit b0d7c4c into dev May 13, 2025
16 checks passed
@eriklharper eriklharper deleted the eriklharper/12122-windows-input-time-picker-color-contrast branch May 13, 2025 16:05
benelan pushed a commit that referenced this pull request May 14, 2025
… input is focused (#12130)

**Related Issue:** #12122 

## Summary

This inverts the text color of each focused input in
`calcite-input-time-picker` for Windows browsers whose system highlight
color doesn't provide sufficient contrast by setting the text color to
use the system `HighlightText` keyword.

For more information: https://css-tricks.com/system-things/

Tested on Chrome, Firefox and Edge on Windows 10 Enterprise build
19045.5737.

Chrome Version 136.0.7103.93 (Official Build) (64-bit)
<img width="562" alt="image"
src="https://github.com/user-attachments/assets/e883157e-6f93-4d0d-8f8b-63dc6e466e90"
/>

Firefox Version 138.0.1 (64-bit)

![image](https://github.com/user-attachments/assets/6893ef26-c17b-48e6-b303-8c5554b5f10e)

Edge Version 136.0.3240.64 (Official build) (64-bit)

![image](https://github.com/user-attachments/assets/e82c85bb-ef95-4e90-9ef5-b0f79903c4b6)
benelan added a commit that referenced this pull request May 15, 2025
* origin/dev: (277 commits)
  docs(tokens): consistency pass for new component descriptions (#12148)
  build(preset): use valid TS module resolution (#12151)
  docs: update list of contributors (#12134)
  chore: drop obsolete transforms (#12136)
  chore: release main (#11890) (#12147)
  build(deps): bump @arcgis/lumina, typescript, vite, vitest (#12137)
  chore(preset): fix JSON import in build config (#12142)
  docs(panel, action): update `text-color-pressed` token descriptions (#12140)
  chore: release next
  fix(input-time-picker): invert text color on Windows when each masked input is focused (#12130)
  chore: release next
  refactor(sematic-tokens): update `--calcite-corner-radius-default` to reference correct token (#12131)
  feat(semantic-tokens): add `--calcite-color-text-highlight` tokens (#12068)
  chore: release next
  feat(combobox): add `selectAll` toggle property (#11721)
  chore: release next
  fix(sort-handle): allow move and reorder events to be cancelled (#12095)
  chore: release next
  feat: added spike data, heart chart, and progress bar icons (#12127)
  feat(text-area): Add design tokens for corner radius, shadow, footer background color (#12124)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug reports for broken functionality. Issues should include a reproduction of the bug. pr ready for visual snapshots Adding this label will run visual snapshot testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants