Skip to content
This repository has been archived by the owner on Aug 21, 2023. It is now read-only.

Input: Fix webkit autocomplete styles #101

Merged
merged 2 commits into from
Oct 26, 2017
Merged

Conversation

ItsJonQ
Copy link
Contributor

@ItsJonQ ItsJonQ commented Oct 26, 2017

Input: Fix webkit autocomplete styles

Chrome

screen shot 2017-10-26 at 10 34 01 am

Safari

screen shot 2017-10-26 at 10 35 14 am

This update adjusts the CSS for Input (and indirectly, Select), to allow for
the webkit autocomplete state to render correctly.

Previously, the autocomplete styles were overlapping the Input component's
"Backdrop" layer (a technique used to render the appearance/border of the
component). To resolve this, I've added a 1px padding around the container,
which simulates the offset a typical non-backdrop border would provide.

With this change, the autocomplete styles no-longer overflow on top of
the backdrop layer.

This update is purely a CSS change. The only JS changes made were to update
the Input component Stories to add an autocomplete example (and to refactor).

馃憦

These changes were manually visually tested in Chrome + Safari. Also tested in Firefox for regressions.

Resolves: #100

This update adjusts the CSS for Input (and indirectly, Select), to allow for
the webkit autocomplete state to render correctly.

Previously, the autocomplete styles were overlapping the Input component's
"Backdrop" layer (a technique used to render the appearance/border of the
component). To resolve this, I've added a 1px padding around the container,
which simulates the offset a typical non-backdrop border would provide.

With this change, the autocomplete styles no-longer overflow on top of
the backdrop layer.

This update is purely a CSS change. The only JS changes made were to update
the Input component Stories to add an autocomplete example (and to refactor).

:clapping:
@ItsJonQ ItsJonQ self-assigned this Oct 26, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 4d1254d on input/fix-autocomplete-styles into d8382b7 on master.

@ItsJonQ ItsJonQ merged commit 5e3999f into master Oct 26, 2017
@ItsJonQ ItsJonQ deleted the input/fix-autocomplete-styles branch October 27, 2017 19:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Input: Autocomplete style issues
2 participants