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

When editing title of screenshot, highlight the entire word. #4182

Closed
sevaan opened this issue Mar 5, 2018 · 0 comments
Closed

When editing title of screenshot, highlight the entire word. #4182

sevaan opened this issue Mar 5, 2018 · 0 comments
Assignees
@punamdahiya
Labels
enhancement good first issue skill:js ux polish
Milestone
Sprint 9 (61-1) 👋

Comments

@sevaan
Copy link
Collaborator

sevaan commented Mar 5, 2018

Currently, when editing a Screenshot’s title, the cursor is placed before all the text forcing the user to do extra work to erase the title or append something to the end.

screen shot 2018-03-05 at 10 17 37 am

Fix: We should highlight the entire text so when a user types it overrides everything (or they can arrow left or right to place the cursor where they need it).
@ghost ghost added this to the Sprint 10 (61-2) ⛅ milestone Mar 20, 2018
@ghost ghost assigned punamdahiya Mar 20, 2018
@ghost ghost modified the milestones: Sprint 10 (61-2) ⛅, Sprint 9 (61-1) 👋 Mar 20, 2018
@chenba chenba closed this as completed in 86b0fd6 Mar 23, 2018
chenba added a commit that referenced this issue Mar 23, 2018
@chenba
Merge pull request #4259 from punamdahiya/4182-edit-title-highlight
ad67c21 
Fixes #4182 - Highlight entire word when editing title
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Assignees

@punamdahiya punamdahiya

Labels
enhancement good first issue skill:js ux polish
Projects
None yet
Milestone
Sprint 9 (61-1) 👋
Development

No branches or pull requests
4 participants
@ianb @sevaan @punamdahiya @johngruen