[userstory] Implement new Downloads Panel item state for items that can be unblocked

NEW
Unassigned

Status

()

defect
5 years ago
a year ago

People

(Reporter: Paolo, Unassigned)

Tracking

(Depends on 5 bugs)

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(relnote-firefox -)

Details

(Whiteboard: [fxprivacy] [userstory])

Comment hidden (empty)
(Reporter)

Updated

5 years ago
Blocks: 1062966
Summary: Implement new Downloads Panel item state for items that can be unblockable → Implement new Downloads Panel item state for items that can be unblocked
(Reporter)

Updated

5 years ago
OS: Windows 7 → All
Hardware: x86_64 → All
(Reporter)

Comment 1

5 years ago
This bug is about the part of the work defined by bug 1062966 related to adding the new panel item state for downloads that can be unblocked. This is just the item itself, the confirmation dialog can be implemented separately in bug 1068660.

This should be based on the mockups in bug 1053890 and their evolution, as well as the strings in bug 1063105.
Blocks: 1068660
(Reporter)

Updated

5 years ago
Depends on: 1068664
(Reporter)

Updated

5 years ago
Points: --- → 5
Flags: firefox-backlog+
Flags: qe-verify?
(Reporter)

Updated

5 years ago
Flags: qe-verify? → qe-verify+
(Reporter)

Updated

5 years ago
See Also: → 1073598
Assignee: nobody → mhammond
Status: NEW → ASSIGNED
Iteration: --- → 35.3

Updated

5 years ago
QA Contact: catalin.varga
I'm not going to get to this in this iteration.
Assignee: mhammond → nobody
Status: ASSIGNED → NEW
Iteration: 35.3 → ---
Is there anyone who can take this bug? This is blocking turning on remote lookups for application reputation, which accounts for approximately half of malware classifications.
Flags: needinfo?(gavin.sharp)
We've been blocked on bug 1068664 for a while. We'll figure something out this week.
Flags: needinfo?(gavin.sharp)
Gavin has asked me to take this. I'm not sure if I'll get to it in 37.1. It may slip until 37.2. Marco, can you please add this for the 37.1 iteration?
Assignee: nobody → jaws
Status: NEW → ASSIGNED
Iteration: --- → 37.1
Flags: needinfo?(mmucci)
Added to IT 37.1
Flags: needinfo?(mmucci)
Iteration: 37.1 → 37.2
(Reporter)

Updated

4 years ago
Keywords: relnote
(Reporter)

Comment 7

4 years ago
Jared, I took a closer look at the front-end code, and now this bug seems to me to be quite more difficult than I though initially, since the changes must be done for both the Downloads Panel and the Downloads View in the Library, or in content for private windows. Most things are currently implemented separately.

Also, we currently reduce the download state to an integer. This is a legacy from the old interface, and this patch would need to look at other state instead, requiring a deeper CSS refactor.

I'm looking into possible simplifications. Since I know you're working on other priorities at this time, I might have something ready before you even start working on this bug. However, I believe this simplification work shouldn't stop you from getting a preliminary version out with the current state.

You may see some related review requests from me in the next few days, but there is no hurry to handle them.
Points: 5 → 13
(In reply to :Paolo Amadini from comment #7)
> the Downloads View in the
> Library, or in content for private windows.

These 2 views are managed by the same code (allDownloadsViewOverlay), only the surrounding code changes (browser/components/places/content/downloadsViewOverlay.xul or browser/components/downloads/content/contentAreaDownloadsView.xul)
Iteration: 37.2 → 37.3
(In reply to :Paolo Amadini from comment #7)
> I'm looking into possible simplifications. Since I know you're working on
> other priorities at this time, I might have something ready before you even
> start working on this bug. However, I believe this simplification work
> shouldn't stop you from getting a preliminary version out with the current
> state.

I've actually been working on this now for a couple days, and I agree that the task is much larger than I initially assumed. I'm thinking of ways that this bug may be broken down.
(Reporter)

Updated

4 years ago
Depends on: 1115364
(Reporter)

Comment 10

4 years ago
(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #9)
> I've actually been working on this now for a couple days, and I agree that
> the task is much larger than I initially assumed. I'm thinking of ways that
> this bug may be broken down.

First things first, I've made the code style consistent in bug 1115364 - will make code easier to read and move around across files.

I'm using dependencies of this bug to track my work, if it turns out your work spans more than one bug as well, we may actually turn this into a meta-bug.
(Reporter)

Updated

4 years ago
Depends on: 1115369
(Reporter)

Updated

4 years ago
Depends on: 1115379
(Reporter)

Updated

4 years ago
Depends on: 1115421
(Reporter)

Updated

4 years ago
Depends on: 1115971
(Reporter)

Updated

4 years ago
Depends on: 1115972
(Reporter)

Updated

4 years ago
Depends on: 1115983
(Reporter)

Updated

4 years ago
Depends on: 1116176
(Reporter)

Updated

4 years ago
Depends on: 1117139
(Reporter)

Updated

4 years ago
Depends on: 1117141
(Reporter)

Updated

4 years ago
Depends on: 1117145
Duplicate of this bug: 1066133
We don't use the relnote keyword, changing to the relnote-firefox ? flag so this will be noted when it's fixed.
relnote-firefox: --- → ?
Keywords: relnote
This bug is currently stalled until the refactoring work is complete. When this bug is picked back up, I'll break it down in to smaller chunks that will be easier to implement and review.
Iteration: 37.3 - 12 Jan → ---
(Reporter)

Updated

4 years ago
Depends on: 1127867
Paolo, can you drive this bug to completion? The patch that I have so far can be found at http://hg.mozilla.org/users/jwein_mozilla.com/jaws/file/cadb078649a7/1068656.patch, but it is not based off of your latest refactorings.
Assignee: jaws → nobody
Status: ASSIGNED → NEW
Flags: needinfo?(paolo.mozmail)
(Reporter)

Comment 15

4 years ago
(In reply to Jared Wein [:jaws] (please needinfo? me) from comment #14)
> Paolo, can you drive this bug to completion? The patch that I have so far
> can be found at
> http://hg.mozilla.org/users/jwein_mozilla.com/jaws/file/cadb078649a7/1068656.
> patch, but it is not based off of your latest refactorings.

Thanks for the reference, I'll work on updating the refactorings this week, then I can use your patch as a starting point.
Flags: needinfo?(paolo.mozmail)
(Reporter)

Updated

4 years ago
Depends on: 1129896
Hi Paolo and Jared,

Any update on this bug?  Looks like there is only once dependency that is still open - https://bugzilla.mozilla.org/show_bug.cgi?id=1117145
(In reply to Tanvi Vyas [:tanvi] from comment #16)
> Hi Paolo and Jared,
> 
> Any update on this bug?  Looks like there is only once dependency that is
> still open - https://bugzilla.mozilla.org/show_bug.cgi?id=1117145

Hi Tanvi, Paolo and I have met and discussed what would be necessary to move this bug forward. We will be filing a bug blocking this one to outline the steps.
(Reporter)

Updated

4 years ago
Depends on: 1139472
(Reporter)

Updated

4 years ago
Depends on: 1139913
(Reporter)

Updated

4 years ago
Depends on: 1139914
(Reporter)

Updated

4 years ago
Blocks: 1019933
Whiteboard: [fxprivacy]
Whiteboard: [fxprivacy] → [fxprivacy] [triage]
Points: 13 → ---
Flags: qe-verify+
Flags: firefox-backlog+
QA Contact: catalin.varga
Whiteboard: [fxprivacy] [triage] → [fxprivacy] [userstory]
Summary: Implement new Downloads Panel item state for items that can be unblocked → [userstory] Implement new Downloads Panel item state for items that can be unblocked
(Reporter)

Updated

4 years ago
Depends on: 1198181
(Reporter)

Comment 18

4 years ago
I thought I had already filed the final item implementation as a separate bug, but this one was probably intended to be the implementation one originally, so I filed bug 1198181 instead.
(Reporter)

Updated

3 years ago
Depends on: 1241177
(Reporter)

Updated

3 years ago
Depends on: 1244473
(Reporter)

Updated

3 years ago
Depends on: 1254100
Depends on: 1253585
(Reporter)

Updated

3 years ago
Depends on: 1265335
(Reporter)

Updated

3 years ago
Depends on: 1265338
(Reporter)

Updated

3 years ago
Depends on: 1265339
(Reporter)

Updated

3 years ago
Depends on: 1265341
(Reporter)

Updated

3 years ago
Depends on: 1265344
(Reporter)

Updated

3 years ago
Depends on: 1265358
(Reporter)

Updated

3 years ago
Depends on: 1265359
(Reporter)

Updated

3 years ago
Depends on: 1265362
(Reporter)

Updated

3 years ago
Depends on: 1265363
(Reporter)

Updated

3 years ago
Depends on: 1265382
(Reporter)

Updated

3 years ago
Depends on: 1265384
(Reporter)

Updated

3 years ago
Depends on: 1265387
(Reporter)

Updated

3 years ago
Depends on: 1265391
(Reporter)

Updated

3 years ago
No longer blocks: 1019933
Doing some release note flag cleanup. This was released in Firefox 48 and wasn't relnoted.
You need to log in before you can comment on or make changes to this bug.