Steps to reproduce: 0. have javascript turned on 1. go to an open bug 2. change e-mail address in 'assignee' box just above 'commit' 3. notice that js automatically selects the 'reassign' box for you (good) 4. change your mind, and select "resolve bug" instead 5. clear e-mail address for 'reassign', or change back to original owner, because you're worried that you might change something accidentally. 6. notice that js automatically selects the 'reassign' box for you (bad) This is somewhere between minor and enhancement and "future me!!!".

Same for resolve as duplicate.

Although not impossible to solve, this would be a fair bit of work compared to the existing code. Right now it works with a very simple "onChange" handler. If you change that field, it selects it. It doesn't care what you change it to. To pull this off, you'd have to move the onChange handler into a separate subroutine, instead of having it all right in the INPUT tag. You would also need an onLoad handler that grabbed the initial value of that field. Once it's changed, the onChange handler would need to determine if it matched the initial value or not before deciding to select it.

*** This bug has been marked as a duplicate of 76161 ***

OK, apparently this is harder. We'll leave the duplicate issue over at bug #76161 and keep this separate.

->New product Bugzilla and taking.

Really moving this time :)

I'm not sure what Dave was thinking of in his comments for this with the onload stuff and separate subroutines, but this is a relatively simple patch which sets the value of the assignee email into a JS var (since we're in JS anyway with the onChange() handler), and then compare the newly entered value to the old value. r= from someone? :)

Comment on attachment 47984 [details] [diff] [review] Simpler, better patch. r=kiko looks much better now, harmless fix.

Applied to http://bugs.async.com.br/show_bug.cgi?id=130 So whoever verifies can test the validity of the fix.

Comment on attachment 47985 [details] [diff] [review] Third time's a charm! This also checks if the value is blank. r=bbaetz, and tested on the given bugzilla installation

fix checked in.