Closed
Bug 703347
Opened 13 years ago
Closed 12 years ago
Add a background image for the "void beneath" the web
Categories
(Firefox for Android Graveyard :: General, defect, P3)
Tracking
(firefox11 fixed, firefox12 verified, firefox13 verified, fennec11+)
VERIFIED
FIXED
Firefox 12
People
(Reporter: ibarlow, Assigned: kats)
References
Details
Attachments
(3 files, 4 obsolete files)
32.30 KB,
image/png
|
Details | |
17.88 KB,
image/png
|
Details | |
30.00 KB,
patch
|
cwiiis
:
review+
akeybl
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Right now, if I scroll past the edge of the web view, I see a grey background. It'd be a nice design detail to reveal a subtle background texture, instead. I don't have an image yet, but can make one, if we can do this
Reporter | ||
Comment 1•13 years ago
|
||
Attachment #575259 -
Attachment is obsolete: true
Comment 2•13 years ago
|
||
Let's not do something that causes a performance regression though. The checkerboard in XUL caused a performance regression.
Comment 3•13 years ago
|
||
(In reply to Mark Finkle (:mfinkle) from comment #2) > Let's not do something that causes a performance regression though. The > checkerboard in XUL caused a performance regression. The "void beneath" the web page is not necessarily equivalent to the checkerboard I think. I guess Ian means the area that becomes visible when you scroll beyond the page boundaries. I agree it would be a nice touch if we had a subtle texture there. One that doesn't impact performance, of course :-) (This is P5 material though...)
Updated•13 years ago
|
OS: Mac OS X → Android
Hardware: x86 → ARM
Updated•13 years ago
|
Assignee: nobody → kgupta
Priority: -- → P3
Assignee | ||
Comment 4•13 years ago
|
||
We used to have a background there in the birch-pan-zoom patch queue, but it was taken out. Not sure why, I kind of liked it :)
Assignee | ||
Comment 6•13 years ago
|
||
Do we still want this? Ian: if so, please add the desired texture to this bug and I can add it.
Reporter | ||
Comment 7•13 years ago
|
||
Hey kats, thanks for reminding me! Attached is an image you can try using in the background. Size as, tiled vertically and horizontally please :)
Comment 8•13 years ago
|
||
Please place the texture in "drawable-nodpi/" folder so that it doesnt scale for different dpi's. Also, please rename it to very_dark_grey.png to be consistent with a change I'm planning to make in next few days.
Updated•13 years ago
|
tracking-fennec: --- → 11+
Assignee | ||
Comment 9•12 years ago
|
||
ibarlow: I have a patch for this, but to be honest on my devices the texture isn't really visible, it's too subtle and just looks like a plain dark gray background. I had to draw a bright yellow rectangle on it while testing to verify it was actually getting drawn. Maybe a different texture would work better? I have a build with the texture applied at https://people.mozilla.com/~kgupta/tmp/bgvoid.apk if you want to try it out. Also, I had to resize the texture to 128x128 so that it's a power-of-two size for tiling. I just use OS X Preview to do the resize; if scaling up with that is going to be a problem it would be better to generate the texture at 128x128 originally rather than 100x100.
Reporter | ||
Comment 10•12 years ago
|
||
Hi Kats, thanks for the build. I agree, it's a little too subtle, so I'll make a new, granier image for you (at 128x128) and post it here shortly.
Updated•12 years ago
|
status-firefox11:
--- → affected
status-firefox12:
--- → affected
Reporter | ||
Comment 11•12 years ago
|
||
Hi Kats, here's an updated image that has a little more contrast applied, let's see if this looks better. It's also been sized up to 128x128
Attachment #582925 -
Attachment is obsolete: true
Assignee | ||
Comment 12•12 years ago
|
||
Attachment #588207 -
Attachment is obsolete: true
Attachment #589817 -
Flags: review?(chrislord.net)
Assignee | ||
Comment 13•12 years ago
|
||
Same patch, but now also contains the binary for the image file (my git show alias was missing the --binary flag).
Attachment #589817 -
Attachment is obsolete: true
Attachment #589817 -
Flags: review?(chrislord.net)
Attachment #589818 -
Flags: review?(chrislord.net)
Comment 14•12 years ago
|
||
Comment on attachment 589818 [details] [diff] [review] Apply background image Review of attachment 589818 [details] [diff] [review]: ----------------------------------------------------------------- r+, but I worry that this causes significant over-draw without any clipping. We probably needn't worry, given how simple our scene is though.
Attachment #589818 -
Flags: review?(chrislord.net) → review+
Assignee | ||
Comment 15•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/fd479cee8771
Target Milestone: --- → Firefox 12
Comment 16•12 years ago
|
||
This cleanup seems nice and all, but I still don't see why we need it at all. You can even pan the page off into this void of space.. it looks gross, left, right or top. I understand if the checker boarding happens at the bottom of a page, but it just looks like a webpage in space and its not even zoomed out. FWIW, The stock browser doesn't have this outer space beyond the webpage, and its one of the reasons I cannot stand to use fennec. It seems were avoiding the real problem by covering up the terrible UX with a texture.
Comment 17•12 years ago
|
||
(In reply to Dennis "Dale" Y. [:cuz84d] from comment #16) > This cleanup seems nice and all, but I still don't see why we need it at > all. You can even pan the page off into this void of space.. it looks > gross, left, right or top. I understand if the checker boarding happens at > the bottom of a page, but it just looks like a webpage in space and its not > even zoomed out. FWIW, The stock browser doesn't have this outer space > beyond the webpage, and its one of the reasons I cannot stand to use fennec. > It seems were avoiding the real problem by covering up the terrible UX with > a texture. Feel free to subscribe to https://bugzilla.mozilla.org/show_bug.cgi?id=705246 to track the progress of this. Our current behaviour matches Samsung phones, but you're right, is incorrect for most every other Android phone if we want to feel 'native'. I don't think I'd consider it *terrible* UX, however - iPhone users don't seem to mind and there are an awful lot of them.
Comment 18•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/fd479cee8771
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 19•12 years ago
|
||
Comment on attachment 589818 [details] [diff] [review] Apply background image [Approval Request Comment] Regression caused by (bug #): none User impact if declined: minimal; we show a gray background instead of a textured image Testing completed (on m-c, etc.): on m-c Risk to taking this patch (and alternatives if risky): possibly a minor performance regression during paint
Attachment #589818 -
Flags: approval-mozilla-aurora?
Comment 20•12 years ago
|
||
Comment on attachment 589818 [details] [diff] [review] Apply background image [Triage Comment] Mobile only - approved for Aurora.
Attachment #589818 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 22•12 years ago
|
||
Verified fixed on: Nightly Fennec 13.0a1 (2012-03-06) Aurora Fennec 12.0a2 (2012-03-06) Device: HTC Desire Z OS: Android 2.3.3
Status: RESOLVED → VERIFIED
Updated•12 years ago
|
status-firefox13:
--- → verified
Updated•12 years ago
|
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•