unee-t / frontend

Meteor front end
https://case.dev.unee-t.com/
GNU Affero General Public License v3.0
9 stars 17 forks source link

feat: select all text on focus; fix empty val not restored after reopen #865

Closed nbiton closed 5 years ago

nbiton commented 5 years ago

Resolves #822 (but in an alternative way) The entire field's contents are selected when the field is focused, so any character typed would remove the existing text, unless another tap is made on the field to place the caret in a specific location. Also fixes an issue where the solution/next steps value wouldn't be restored if the value was changed, not saved, and the dialog was reopened. image

franck-boullier commented 5 years ago

Test done:

On Heroku App logged in as Nath case 75479

Expected result:

In the More Information page See

Issue:

Instead of Next Step Date we see Deadline.

image

Edit solution:

Expected result:

Actual result:

The text is NOT highlighted with FF on a DT image

Edit Deadline

Edit Next Step:

Expected result:

Actual result:

The text is NOT highlighted with FF on a DT image

Edit Next Step Date:

nbiton commented 5 years ago

The text is NOT highlighted with FF on a DT

Fixed in the last commit

Text reads Deadline instead of Next Step Date FAILED

Since this was already introduced in a previously merged PR, and not as a part of this one, I'll issue another PR to fix it.