implement dirty select tracking in bug-modal to address firefox refresh issue

RESOLVED FIXED

Status

()

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: Fallen, Assigned: glob)

Tracking

Production

Details

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
So I am not totally sure if this is an issue with the new UI, but I don't often run into situations where I make bug changes without knowing about it and I've just been test driving the new UI.

In bug 1154521 comment 22 I made some changes to tracking flags I wasn't aware of. I had this tab open for a while, reloaded the page to get the latest changes, then made my comment and set needinfo. The tracking flags were reset to their previous state.

If there is no good way to track this down feel free to RESO INCOMPLETE, if you can and its really a bug that can be fixed even better :)
(Assignee)

Comment 1

3 years ago
looks like i forgot about bug 437212.
Assignee: nobody → glob
Blocks: 1150541
Summary: Tracking flags are updated without interaction → implement dirty select tracking in bug-modal to address firefox refresh issue
(Assignee)

Comment 2

3 years ago
Created attachment 8604122 [details] [diff] [review]
1163326_1.patch

- implement dirty field tracking

see https://bugzilla.mozilla.org/show_bug.cgi?id=437212#c28 for steps-to-reproduce, except test using a tracking flag.
Attachment #8604122 - Flags: review?(dylan)
Comment on attachment 8604122 [details] [diff] [review]
1163326_1.patch

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

r=dylan

Tested -- dirty tracking flags keep their value (unless they've been changed by another user first, I assume that's intentional).

I used firefox on two different machines to confirm, I was not able to use chrome to test this. I actually am not certain tracking flags can even be set in chrome. No errors in the dev console, just silently failed. Weird, eh?
Attachment #8604122 - Flags: review?(dylan) → review+
(Assignee)

Comment 4

3 years ago
(In reply to Dylan William Hardison [:dylan] from comment #3)
> I actually am not certain tracking flags can even be
> set in chrome. No errors in the dev console, just silently failed. Weird, eh?

filed bug 1164321 for the chrome issue.


To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
   53af86d..0622523  master -> master
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.