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

UI: TextField Fix the cursor #85

Merged
merged 1 commit into from
Sep 29, 2024
Merged

Conversation

ksharma-xyz
Copy link
Owner

@ksharma-xyz ksharma-xyz commented Sep 29, 2024

Fixes

  • Cursor always displayed at the start of Placeholder
  • Cursor will be displayed as soon as the field is focused.

Problem:
When using BasicTextField directly, the cursor is only shown by default when there's text or during text input, i.e when it is in focused state.

We need to ensure that the cursor is always visible when the TextField is in focused state, this means displaying the cursor at the beginning even when the placeholder text is displayed.

Before - No cursor by default

NoCursor.by.default.mov

After

Cursor_After.mov

Copy link
Owner Author

ksharma-xyz commented Sep 29, 2024

@ksharma-xyz ksharma-xyz force-pushed the 09-29-ui_textfield_fix_the_cursor branch from 145708a to 43bfe42 Compare September 29, 2024 11:17
@ksharma-xyz ksharma-xyz added UI Design System Related changes achievement When a big problem is fixed or something exciting gets working labels Sep 29, 2024
@ksharma-xyz ksharma-xyz marked this pull request as ready for review September 29, 2024 11:38
@ksharma-xyz ksharma-xyz force-pushed the 09-29-ui_textfield_fix_the_cursor branch from 43bfe42 to dbb82e8 Compare September 29, 2024 11:38
Base automatically changed from 09-29-ui_textcolor to main September 29, 2024 22:01
Cursor always displayed at the start of Placeholder
Cursor dislayed as soon as the field is focused.
@ksharma-xyz ksharma-xyz force-pushed the 09-29-ui_textfield_fix_the_cursor branch from dbb82e8 to 85608e5 Compare September 29, 2024 22:08
@ksharma-xyz ksharma-xyz enabled auto-merge (squash) September 29, 2024 22:09
@ksharma-xyz ksharma-xyz merged commit 5fcc7d6 into main Sep 29, 2024
1 check passed
@ksharma-xyz ksharma-xyz deleted the 09-29-ui_textfield_fix_the_cursor branch September 29, 2024 22:13
ksharma-xyz added a commit that referenced this pull request Sep 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
achievement When a big problem is fixed or something exciting gets working UI Design System Related changes
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant