Skip to content

Commit

Permalink
fixes #4743 (#4797)
Browse files Browse the repository at this point in the history
fixes #4743 
<!-- Link to relevant issue (for ex: "fixes #1234") which will
automatically close the issue once the PR is merged -->

## PR Type
<!-- Please uncomment one ore more that apply to this PR -->

Bugfix
<!-- - Feature -->
<!-- - Code style update (formatting) -->
<!-- - Refactoring (no functional changes, no api changes) -->
<!-- - Build or CI related changes -->
<!-- - Documentation content changes -->
<!-- - Sample app changes -->
<!-- - Other... Please describe: -->


## Describe the current behavior?
<!-- Please describe the current behavior that is being modified or link
to a relevant issue. -->
When the input was autofilled, the color was a bright blue that did not
have good contrast.

## Describe the new behavior?
It now uses our standard input color.

## PR Checklist

- [ ] Test: run `npm run test` and ensure that all tests pass
- [ ] Target main branch (or an appropriate release branch if
appropriate for a bug fix)
- [ ] Ensure that your contribution follows [standard accessibility
guidelines](https://docs.microsoft.com/en-us/microsoft-edge/accessibility/design).
Use tools like https://webhint.io/ to validate your changes.


## Additional Information
  • Loading branch information
jgw96 authored Aug 27, 2024
1 parent 20b99fa commit 535d3ad
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions apps/pwabuilder/src/script/pages/app-home.ts
Original file line number Diff line number Diff line change
Expand Up @@ -164,6 +164,9 @@ export class AppHome extends LitElement {
display: flex;
flex-direction: column;
}
#input-box::part(input) {
-webkit-text-fill-color: var(--sl-input-color);
}
#start-button {
grid-area: 1 / 5 / auto / auto;
width: 100%;
Expand Down

0 comments on commit 535d3ad

Please sign in to comment.