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

fix(textfield): render prefix before input #161

Merged
merged 1 commit into from
Nov 15, 2023
Merged

Conversation

BalbinaK
Copy link
Contributor

Fixes WARP-337

The prefix should be announced before the input, and suffix - after the input.
Exemplary input with prefix and suffix
Inspected input with prefix and suffix button with correct semantic order
Inspected input with prefix and suffix with correct semantic order

The prefix should be announced before the input, and suffix - after the input.
@BalbinaK BalbinaK requested a review from a team November 15, 2023 12:01
Copy link
Contributor

github-actions bot commented Nov 15, 2023

PR Preview Action v1.4.4
Preview removed because the pull request was closed.
2023-11-15 12:32 UTC

@BalbinaK BalbinaK merged commit 2017cff into next Nov 15, 2023
3 checks passed
@BalbinaK BalbinaK deleted the fix/a11y-textfield branch November 15, 2023 12:31
github-actions bot pushed a commit that referenced this pull request Nov 15, 2023
## [1.2.1-next.2](v1.2.1-next.1...v1.2.1-next.2) (2023-11-15)

### Bug Fixes

* **textfield:** render prefix before input ([#161](#161)) ([2017cff](2017cff))
github-actions bot pushed a commit that referenced this pull request Nov 29, 2023
## [1.2.1](v1.2.0...v1.2.1) (2023-11-29)

### Bug Fixes

* **combobox:** don't open on modifier key press and fix aria-expanded ([#152](#152)) ([71e42f6](71e42f6))
* Import icons in the correct way ([#169](#169)) ([f279db9](f279db9))
* **textfield:** allow styling input's left padding when prefix is wider than 40px ([#173](#173)) ([89e5a27](89e5a27))
* **textfield:** render prefix before input ([#161](#161)) ([2017cff](2017cff))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants