fix: Text input cursor jumps to first position when writing long text

AshishBarvaliya commented 1 year ago

New Pull Request Checklist

[x] I am not disclosing a vulnerability.
[x] I am creating this PR in reference to an issue.

Issue Description

Updated TextInput Component

Closes: #2412

Approach

set selection range when focusing Textarea.

TODOs before merging

parse-github-assistant[bot] commented 1 year ago

Thanks for opening this pull request!

🎉 We are excited about your hands-on contribution!

parse-github-assistant[bot] commented 1 year ago

I will reformat the title to use the proper commit message syntax.

uffizzi-cloud[bot] commented 1 year ago

Uffizzi Ephemeral Environment `deployment-21718`

:watch: Updated Apr 09, 2023, 22:44 UTC

:cloud: https://app.uffizzi.com/github.com/parse-community/parse-dashboard/pull/2413

:page_facing_up: View Application Logs etc.

What is Uffizzi? Learn more

parseplatformorg commented 1 year ago

🎉 This change has been released in version 5.2.0-alpha.6

parseplatformorg commented 1 year ago

🎉 This change has been released in version 5.2.0-beta.2

mathieulb commented 1 year ago

My team is on multiple browsers/OSes and we have a similar bug with beta2 and alpha27, but instead it jumps to the end of the text. It seems like the bug I see is the fix for issue 2413. What I do :

Click in the middle of short text
Type enough text to make Dashboard switch to the layout of long text

What happens :

Cursor jumps to the end
Undo history is cleared
another issue making it worse when can't undo : can't cancel either, as the Escape key does not do anything, there is no Cancel button, and any blur (un-focus) event saves the mistake I made, and even closing the tab saves it. The only way out is to press reload (with or without shift). No way to revert to any previous value.

If undo history can't be carried between two different widgets that you are swapping (INPUT to TEXTAREA and back), then surely the whole concept of swapping widgets is the wrong solution to whatever problem it is trying to solve.

mtrezza commented 1 year ago

@mathieulb could you open a new issue and fill out the template.

parseplatformorg commented 1 year ago

🎉 This change has been released in version 5.2.0

parse-community / parse-dashboard