Closed Bug 880393 Opened 11 years ago Closed 11 years ago

ListView rows in about:home should fade, and not ellipsize

Categories

(Firefox for Android Graveyard :: Theme and Visual Design, defect)

ARM
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 26

People

(Reporter: sriram, Assigned: sriram)

References

Details

(Whiteboard: fixed-fig)

Attachments

(1 file, 1 obsolete file)

ListView rows in about:home should not ellipsize the text. They should instead fade at the end.
Assignee: nobody → sriram
Attached patch Patch (obsolete) — Splinter Review
This adds a faded text view with a new attribute "fadeWith". There are other views in about:home (like the top sites grid view) that also uses fading, but of different size. This can be used there as well.

Somehow setting singleLine and ellipsize in styles.xml is not getting into the TextView. I'm investigating that. Meanwhile, those attributes are added to layout files directly, and this patch (works and) is safe to land.
Attachment #759343 - Flags: review?(mark.finkle)
The last color here should be 0x0 (transparent)

 new int[] { color, color, 0xFFFFFFFF},

I can change it when I push it.
Attached patch PatchSplinter Review
Fixed the styling issue and the transparency.
The styling issue was because we had a values-16 folder, which I didn't see.
Attachment #759410 - Flags: review?(mark.finkle)
Attachment #759343 - Attachment is obsolete: true
Attachment #759343 - Flags: review?(mark.finkle)
Comment on attachment 759410 [details] [diff] [review]
Patch

Looks fine, but I always worry about performance of rendering gradients.
Attachment #759410 - Flags: review?(mark.finkle) → review+
https://hg.mozilla.org/mozilla-central/rev/2b2bdaae5bd3
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: