1162854 - hitting back after making an edit should keep edit mode open

Status: RESOLVED FIXED

Description

[Byron Jones ‹:glob› 🎈]

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

Comment 1

[Byron Jones ‹:glob› 🎈]

Created attachment 8603198 [details] [diff] [review]
1162854_1.patch

Comment 2

[David Lawrence [:dkl]]

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

Comment 3

[Byron Jones ‹:glob› 🎈]

Created attachment 8606854 [details] [diff] [review]
1162854_2.patch

- prevent user-story from stealing focus when navigating back

Comment 4

[David Lawrence [:dkl]]

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

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

r=dkl

Comment 5

[Byron Jones ‹:glob› 🎈]

To ssh://[e-mail]/webtools/bmo/bugzilla.git
   7810a12..ad9f924  master -> master