Closed Bug 1319732 Opened 4 years ago Closed 4 years ago

Side by side add-on installation pop-ups

Categories

(Firefox :: Site Identity, defect, P1)

53 Branch
defect

Tracking

()

VERIFIED FIXED
Firefox 53
Tracking Status
firefox50 --- unaffected
firefox51 --- unaffected
firefox52 --- unaffected
firefox53 --- verified

People

(Reporter: vasilica.mihasca, Assigned: johannh)

References

Details

(Whiteboard: [fxprivacy] )

Attachments

(2 files)

[Affected versions]:
Firefox 53.0a1 (2016-11-22)


[Affected platforms]:
Windows 10 64-bit
Ubuntu 16.04 32-bit


[Steps to reproduce]:
1.Launch Firefox with clean profile.
2.Navigate to https://addons.mozilla.org
3.Choose an add-on and click on “+ Add to Firefox” green button. 
4.Complete the installation process by clicking on “Install” button from doorhanger.
5.While the confirmation doorhanger is still displayed, choose another add-on and click on “+ Add to Firefox” green button.


[Expected Results]:
The second add-on doorhanger appears under the first one. 


[Actual Results]:
- The doorhangers are displayed one next each other.
- UI issues: http://screencast.com/t/qTqmPh6zZyOK
Whiteboard: [fxprivacy] [triage]
Whiteboard: [fxprivacy] [triage] → [fxprivacy]
Priority: -- → P1
Duplicate of this bug: 1171636
Attached image Old multiple popups
Philipp, can you please check that the old behavior (see screenshot) is still something we want to do for the new doorhanger design? Meaning should we still stack them on top of each other like before or is there some other concept we could use?
Flags: needinfo?(philipp)
Redirecting this to Markus as per our conversation at the all hands
Flags: needinfo?(philipp) → needinfo?(mjaritz)
I see that they currently stack on top of each other with the download appearing on the right side if it. It would be great if we would stack all on top of each other - including download indicator, but have the most recent install confirmation be on top.

Side note: Our next iteration on the install flow plans to have the install confirmation to be modal so as to not enable multiple installs in parallel but require a decision on whether or not to install an extension before continue using the site.
Flags: needinfo?(mjaritz)
Assignee: nobody → jhofmann
Status: NEW → ASSIGNED
Comment on attachment 8822202 [details]
Bug 1319732 - Stack popupnotifications vertically.

Can we fix bug 1320314 instead? See my proposal in bug 1320314 comment 3.
Attachment #8822202 - Flags: review?(dao+bmo)
Maybe my brain is failing to correctly apply the proposed CSS in bug 1320314 comment 3, would that solve the problem at hand here?

Or did you mean we should fix bug 1320314 before fixing this one?
Flags: needinfo?(dao+bmo)
(In reply to Johann Hofmann [:johannh] from comment #7)
> Maybe my brain is failing to correctly apply the proposed CSS in bug 1320314
> comment 3, would that solve the problem at hand here?

It would allow us to get rid of both overflow and the problematic display:block if this comment is to be believed:

   * by hiding the overflow, and we have to override the "display" property so
   * that the height of the contents is computed correctly in that case. */

> Or did you mean we should fix bug 1320314 before fixing this one?

That too, the code doesn't belong in this file.
Flags: needinfo?(dao+bmo)
As I also noted in bug 1320314, the need to avoid "display: -moz-box;" is likely related to bug 1293242 so it will be unneeded once that is fixed.
Depends on: 1320314
Comment on attachment 8822202 [details]
Bug 1319732 - Stack popupnotifications vertically.

https://reviewboard.mozilla.org/r/101186/#review103810
Attachment #8822202 - Flags: review?(dao+bmo) → review+
Pushed by jhofmann@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f7c9a456f7a1
Stack popupnotifications vertically. r=dao
https://hg.mozilla.org/mozilla-central/rev/f7c9a456f7a1
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 53
I have to reproduced this bug with Nightly 53.0a1 (2016-11-22) on Windows 7 64 bit;

The Bug's fix is verified on beta

Build ID 	20170111030235
User Agent 	Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:53.0) Gecko/20100101 Firefox/53.0

[Bugday-20170111]
I have reproduced this bug with  Nightly 53.0a1 (2016-11-23) on Ubuntu 16.04 LTS , 64 Bit !

This bug's fix is verified with latest Nightly 

Build   ID   20170111030235
User Agent   Mozilla/5.0 (X11; Linux x86_64; rv:53.0) Gecko/20100101 Firefox/53.0
[bugday-20170111]
Thanks!
Status: RESOLVED → VERIFIED
Based on comment 16, mark bug verified on firefox 53.
You need to log in before you can comment on or make changes to this bug.