Webapp installer doorhanger should show a standard icon, not the icon related to the app

VERIFIED FIXED in Firefox 14

Status

Firefox Graveyard
Web Apps
VERIFIED FIXED
5 years ago
a year ago

People

(Reporter: Felipe, Assigned: Felipe)

Tracking

Trunk
Firefox 14
Bug Flags:
in-moztrap +

Details

(Whiteboard: [marketplace-beta?])

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

5 years ago
Created attachment 611018 [details]
app install doorhanger

As I understand it, for spoofing reasons the doorhanger that offers the web app installation should not display the app icon, but rather our standard icon that represents a "web app". Is this correct?

(note: "web app" icon display in this screenshot is not final)
(Assignee)

Comment 1

5 years ago
Created attachment 611020 [details]
app install doorhanger

(previous file was corrupted for some reason)
Attachment #611018 - Attachment is obsolete: true
Jen - Could you provide insight into this?
(Assignee)

Updated

5 years ago
Attachment #611020 - Flags: ui-review?(jboriss)

Updated

5 years ago
Whiteboard: [marketplace-beta?]
(Assignee)

Comment 3

5 years ago
Comment on attachment 611020 [details]
app install doorhanger

Can someone from UX take a look and sign off or comment on this change?
Attachment #611020 - Flags: ui-review?(jboriss) → ui-review?(ux-review)
Brian Dils, could you review this?
My opinion for the UX - I'd either go all or nothing here for the icon. Either include the icon for the app or don't include an icon at all. Putting the box there if the icon is being removed I don't think is a good idea, as I don't think it creates any value-add.
Comment on attachment 611020 [details]
app install doorhanger

The 64x64 icons for the notification panels are designed to be distinct and easily recognized in peripheral vision. See geolocation, password, etc. So the icon is desired.
Attachment #611020 - Flags: ui-review?(ux-review) → ui-review+
(In reply to Stephen Horlander from comment #6)
> Comment on attachment 611020 [details]
> app install doorhanger
> 
> The 64x64 icons for the notification panels are designed to be distinct and
> easily recognized in peripheral vision. See geolocation, password, etc. So
> the icon is desired.

If the icon is desired, then I would suggest a different icon to be used. The open box does not provide context to a user that they are installing an app. We need an icon that provides better context that an app is being installed.
The icon for app installation is happening in bug 735680.
(Assignee)

Comment 9

5 years ago
Created attachment 614314 [details] [diff] [review]
Patch
Attachment #614314 - Flags: review?(dao)

Updated

5 years ago
Attachment #614314 - Flags: review?(dao) → review+
FYI - This bug does not fall in alignment with requirements in bug 738546. Please get clarity from product management on the direction for this, as fennec native & firefox need to be alignment on implementation for the icon on app install.
It's not clear who exactly you mean by "product management", but they do not have final veto power on what we end up doing. This is a collaboration, so "product management" needs to engage with the visual design team and the engineers fixing these bugs to determine the best solution. That goes both ways.
(In reply to Gavin Sharp (use gavin@gavinsharp.com for email) from comment #11)
> It's not clear who exactly you mean by "product management", but they do not
> have final veto power on what we end up doing. This is a collaboration, so
> "product management" needs to engage with the visual design team and the
> engineers fixing these bugs to determine the best solution. That goes both
> ways.

I'm referring to Jen & Ragavan. Yeah, I agree collaboration needs to take place. The underlying concern I'm trying to watch out for here is consistency, given that the fennec native implementation is going to have the app icon on a prompt for install, but this bug is suggesting to do the opposite. We should aim for consistency across each implementation, given that falls in line with the apps vision for consistency.

Updated

5 years ago
Whiteboard: [marketplace-beta?] → pmneeded [marketplace-beta?]
While I am certainly a proponent of cross product consistency, we also need to be consistent internally. In this case that means this should look and act like other notification panels. Which means a clear and consistent notification icon, notification text and button labeled with the action to take.

If we need to also include the icon of the application that is asking to be installed, that's ok, it just can't take the place of the notification icon. So if that is a requirement then we have more design work to do to figure out how to include that.
(In reply to Jason Smith from comment #12)

> I'm referring to Jen & Ragavan. Yeah, I agree collaboration needs to take
> place. The underlying concern I'm trying to watch out for here is
> consistency, given that the fennec native implementation is going to have
> the app icon on a prompt for install, but this bug is suggesting to do the
      --------------------------------
> opposite. We should aim for consistency across each implementation, given
> that falls in line with the apps vision for consistency.

It is? Fennec's notification boxes don't include icons at the moment.

Jason - you're quite right - we should get some agreement on this. Can you post the bug number or wiki page for the mobile designs?
(In reply to Madhava Enros [:madhava] from comment #14)
> It is? Fennec's notification boxes don't include icons at the moment.

Right. It's coming down the pipeline though. See bug 738546.

> Jason - you're quite right - we should get some agreement on this. Can you
> post the bug number or wiki page for the mobile designs?

bug 738546 has some proposed UI designs Harald is working on.

Updated

5 years ago
Whiteboard: pmneeded [marketplace-beta?] → [marketplace-beta?]
FYI - Felipe you are good to go. See bug 738546.
(Assignee)

Comment 17

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/9a4a992d1bef
https://hg.mozilla.org/mozilla-central/rev/9a4a992d1bef
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 14
Verified on Nightly.
Status: RESOLVED → VERIFIED

Updated

5 years ago
Flags: in-moztrap?(jsmith)

Updated

5 years ago
Flags: in-moztrap?(jsmith) → in-moztrap+

Updated

5 years ago
QA Contact: jsmith
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.