Open Bug 453932 Opened 16 years ago Updated 14 years ago

CC edit list is hidden when going back to the page with the back button

Categories

(Bugzilla :: Bugzilla-General, defect)

3.4.6
defect
Not set
normal

Tracking

()

People

(Reporter: justdave, Unassigned)

References

Details

Steps to reproduce:

1. Visit any bug
2. Click the edit link next to the CC list
3. Fill in an email address that doesn't exist and won't match anything
4. Submit
5. View error that says the address doesn't exist and to press Back and fix it.
6. Press the Back button

Actual results:

CC list edit controls are hidden.  Clicking the edit link reveals that your edits in progress (including the mistake) are still there.

Expected results:

Since there were actually pending edits, the CC edit controls should be visible, just like we do with the other fields in this situation.

Filed in other b.m.o issues because I'm not sure if this widget made it upstream yet or not.  If it did, this bug probably ought to move to the Bugzilla product.
Assignee: nobody → ui
Component: Bugzilla: Other b.m.o Issues → User Interface
Product: mozilla.org → Bugzilla
QA Contact: other-bmo-issues → default-qa
Target Milestone: --- → Bugzilla 3.2
Version: other → 3.2
this is a bad bug that cause data loss and lots of confusion. I thought I caught all of them but I must have missed this one. Good catch dave.
Flags: blocking3.2?
bug 453932 hasn't even been checked in upstream yet.
Assignee: ui → nobody
Component: User Interface → Bugzilla: Other b.m.o Issues
Flags: blocking3.2?
Product: Bugzilla → mozilla.org
QA Contact: default-qa → other-bmo-issues
Target Milestone: Bugzilla 3.2 → ---
Version: 3.2 → other
Investigating this bug it seems I must have thrown my hands up about this issue (checking multiple fields to determine show/hide) and here is why I did (even though I shouldn't have). 

I will need to write something that is able to check multiple fields for not matching some original value. In this case I need to check that add is blank, nothing is selected in remove multiselect and the remove cc checkbox isn't selected.

If I write this in a generalized fashion, it should also work for groups and flags as well. I could also write one specifically for the CC list though that just checks that those 3 fields are blank.

Either way, I think we should fix this for 3.2.
Assignee: nobody → ui
Component: Bugzilla: Other b.m.o Issues → Bugzilla-General
Product: mozilla.org → Bugzilla
QA Contact: other-bmo-issues → default-qa
Target Milestone: --- → Bugzilla 3.2
Version: other → 3.2
this is verified on landfill.
Bugzilla 3.2 is restricted to security bugs only. Moreover, this bug is either assigned to nobody or got no traction for several months now. Rather than retargetting it at each new release, I'm clearing the target milestone and the bug will be retargetted to some sensible release when someone starts fixing this bug for real (Bugzilla 3.8 more likely).
Target Milestone: Bugzilla 3.2 → ---
Hm. This is a case when apparently Gecko browsers are better than others:
1. Click "edit" for the CC list of this bug.
2. Add "foobar@domain.invalid" as new CC.
3. Click "Commit".
-- Bugzilla (3.4.6+, mozilla.org implementation) complains that "foobar@domain.invalid" didn't match anything.
4. Click the Back button.
-- In SeaMonkey 2.1a2pre (i.e. trunk nightly) we're back as we were between steps 2 and 3 (GOOD).
-- In Konqueror (4.3.5 release 0) the CC-edit input box is hidden (BAD).
5. (Konqueror) Repeat step 1.
-- (Konqueror) We're back as we were between steps 2 and 3.
Version: 3.2 → 3.4.6
(In reply to comment #2)
> bug 453932 hasn't even been checked in upstream yet.

What did you actually mean here?
You need to log in before you can comment on or make changes to this bug.