1023405 - Shortcut for marking bugs as read

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: MyDashboard, defect, P2)

Description

[Bobby Holley (:bholley)]

Currently, when using the "Updated Since Last Visit" query, I get a summary of the updates in the dashboard (which is awesome!) but have to load the bug to make it disappear from the query. It would be great to have a "mark as read" button that would do something lighter-weight and AJAX-y, and remove the bug from the query.

Comment 1

[(no longer active)]

I was going to file the same bug! :)

Comment 2

[:glob ✱]

were you two listening in to our bmo team meeting last night where we discussed this?

Comment 3

[(no longer active)]

(In reply to Byron Jones ‹:glob› from comment #2)
> were you two listening in to our bmo team meeting last night where we
> discussed this?

Great minds think alike.  ;-)

Comment 4

[:glob ✱]

dylan, you've been wandering in this neck of the woods recently; this would be rather useful, especially after bug 1020558 is deployed.

Comment 6

[Dylan Hardison [:dylan] (he/him)]

I figured out what this should look like today. I will try to have something up "today" (Sep 18th America/New_York time) to generate some feedback

Comment 7

[Dylan Hardison [:dylan] (he/him)]

This was a bit more work than I had figured; rather than just use the bug last visit update code from show bug, I have to build and send the queries using YUI3 on the dashboard page. Adjusting the size of the button within the dashboard query view where it doesn't make the rows too tall or wide is fiddly.

Comment 8

[Dylan Hardison [:dylan] (he/him)]

Attachment: bug-1023405-v1.patch

As discussed, this is the simpler UI that merely adds a "mark visited" link next to refresh. Complete with tricky yui2 dependencies.

Comment 9

[David Lawrence [:dkl]]

Comment on attachment 8493479 [details] [diff] [review]
bug-1023405-v1.patch

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

Looks good. One quick change I would like to see is that when you click "Mark Visited" it becomes normal text and no longer a link so that 
the use knows that something has happened. If the user refreshes the list either by clicking "Refresh" or going to another query and then
coming back, then make it a link again.

Comment 10

[Dylan Hardison [:dylan] (he/him)]

Attachment: bug-1023405-v2.patch

The Mark Visited link is now disabled and re-enabled on refresh.

Comment 11

[David Lawrence [:dkl]]

Comment on attachment 8497941 [details] [diff] [review]
bug-1023405-v2.patch

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

Nice. r=dkl

Comment 12

[Dylan Hardison [:dylan] (he/him)]

To ssh://[e-mail]/webtools/bmo/bugzilla.git
   98284f0..c034487  master -> master