Closed Bug 736444 Opened 12 years ago Closed 12 years ago

replace abouthome/noise.png with reasonably sized copy from about:newtab

Categories

(Firefox :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 14
Tracking Status
firefox13 --- fixed

People

(Reporter: dao, Assigned: dao)

References

Details

(Keywords: memory-footprint, Whiteboard: [qa-])

Attachments

(2 files)

Attached patch part 1Splinter Review
      No description provided.
Attachment #606543 - Flags: review?(mak77)
Attached patch part 2Splinter Review
11 KB -> 2.1 KB
Attachment #606544 - Flags: review?(mak77)
Keywords: footprint
hm, the images look quite different to me, is winstripe one just a "merge" of abouthome one and the background?
No, both images are similarly used in combination with a CSS background color and a gradient. The background colors are actually the same and about:home has a slightly brighter gradient.
Comment on attachment 606544 [details] [diff] [review]
part 2

I don't think there's anything to review here, the changes are quite trivial. Though this needs design review imo, I didn't make that theme so I may miss reasons for a certain gradient compared to another one.
Forwarding to Stephen sounds like the best choice.
Attachment #606544 - Flags: review?(mak77) → review?(shorlander)
Comment on attachment 606543 [details] [diff] [review]
part 1

r+ on the idea to replace this image, the current one is quite too big.
Attachment #606543 - Flags: review?(mak77) → review+
The _CSS_ gradient is slightly brighter in about:home. The images don't contain gradients.
Attachment #606544 - Flags: review?(shorlander) → review+
Attachment #606543 - Flags: approval-mozilla-aurora?
Comment on attachment 606544 [details] [diff] [review]
part 2

trivial copy from about:newtab's noise.png to about:home's noise png. both images are new on aurora.
Attachment #606544 - Flags: approval-mozilla-aurora?
Blocks: 711157
Attachment #606543 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Attachment #606544 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: