:glob ✱ Assignee
	Description • 9 years ago

steps:
- make an invalid edit (eg. try to set the assignee to blank)
- hit save
- read the error message
- hit back

expected:
- bug in edit mode so corrections can be made

actual:
- view mode

	David Lawrence [:dkl]
	Comment 2 • 9 years ago

Comment on attachment 8603198 [details] [diff] [review]
1162854_1.patch

Review of attachment 8603198 [details] [diff] [review]:
-----------------------------------------------------------------

<@dkl> glob: strange. bug 1162854. if i open user story modal, but not focus it, and also open people modal, hit edit, clear assignee, commit. When I hit back it always autofocuses the user story text area
<@glob> dkl, ah, i know what that is.  comment in the review i'll work around it

My guess is maybe to do with below but didn't play around with it:

            // if there's no current user-story, it's a better experience if it's editable by default
            if ($('#cf_user_story').val() === '') {
                $('#user-story-edit-btn').click();
            } 

Otherwise looks good will r+ once the fix is verified.

dkl

	:glob ✱ Assignee
	Comment 3 • 9 years ago

- prevent user-story from stealing focus when navigating back

	David Lawrence [:dkl]
	Comment 4 • 9 years ago

Comment on attachment 8606854 [details] [diff] [review]
1162854_2.patch

Review of attachment 8606854 [details] [diff] [review]:
-----------------------------------------------------------------

r=dkl

	:glob ✱ Assignee
	Comment 5 • 9 years ago

To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
   7810a12..ad9f924  master -> master