The default bug view has changed. See this FAQ.

Enforce maximum icon size for nsIAlertsService's notification

VERIFIED FIXED in mozilla16

Status

()

Toolkit
Themes
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: Felipe, Assigned: Felipe)

Tracking

unspecified
mozilla16
Points:
---

Firefox Tracking Flags

(blocking-kilimanjaro:?, firefox16-)

Details

(Whiteboard: [Desktop WebRT], [qa!])

Attachments

(2 attachments)

(Assignee)

Description

5 years ago
Upon a successful webapp installation, we want to display a notification that an app has been installed. On the notification we display the app icon given to us, which can have any size, thus leading to an amusing result: http://i.imgur.com/kNjdd.png

Dão, how should this be handled? can I add a max-width/height 32x32 (or 48x48) on #alertImage in alert.css?  .alertImageBox has a min-width/height but I don't understand why

Updated

5 years ago
Whiteboard: [Desktop WebRT]
Flagging for tracking-firefox16 and k9o - this bug's implementation blocks the implementation of a k9o blocker for providing feedback on post-install, which therefore makes this a blocker.
blocking-kilimanjaro: --- → ?
tracking-firefox16: --- → ?

Comment 2

5 years ago
This doesn't appear to be a blocker for this feature in FF16 (separate from k9o), so no need to track. We'd definitely consider taking an uplift if/when ready, however.
tracking-firefox16: ? → -
(Assignee)

Comment 3

5 years ago
Created attachment 639830 [details] [diff] [review]
Patch

Set max-width/height to 48px
Assignee: nobody → felipc
Status: NEW → ASSIGNED
Attachment #639830 - Flags: review?(dao)
(Assignee)

Comment 4

5 years ago
Created attachment 639832 [details]
Screenshot with patch applied

Comment 5

5 years ago
hmm, perhaps it should have space around it, instead of being jammed against the top and bottom borders?
(In reply to Dan Walkowski from comment #5)
> hmm, perhaps it should have space around it, instead of being jammed against
> the top and bottom borders?

Makes sense to me. Probably a little bit of padding could help improve the look and feel a bit.
(Assignee)

Comment 7

5 years ago
Let's leave that for a follow-up though as we'll probably do more fine tuning on the notification appearance. And use this bug only for the necessary part to land the feedback notification, i.e. enforcing a maximum size for the icon.

Updated

5 years ago
Component: XUL → Themes
Product: Core → Toolkit

Updated

5 years ago
Attachment #639830 - Flags: review?(dao) → review+

Updated

5 years ago
OS: Windows 7 → All
(Assignee)

Comment 8

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/977ee9208065
Target Milestone: --- → mozilla16
https://hg.mozilla.org/integration/mozilla-inbound/rev/20a09c21d3d5
https://hg.mozilla.org/integration/mozilla-inbound/rev/6bfce37d7fa4

Updated

5 years ago
Whiteboard: [Desktop WebRT] → [Desktop WebRT], [qa+]

Updated

5 years ago
QA Contact: jsmith
(Assignee)

Comment 10

5 years ago
http://hg.mozilla.org/mozilla-central/rev/827cfcf5a41b
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
Status: RESOLVED → VERIFIED
Whiteboard: [Desktop WebRT], [qa+] → [Desktop WebRT], [qa!]
You need to log in before you can comment on or make changes to this bug.