631926 - addons with long descriptions does not wrap

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Panning/Zooming, defect)

Description

[Tony Chung [:tchung]]

Attachment: Description clipped and breaks panning screenshot

If a addon has a long description, the text will not properly wrap and align in portrait or landscape view.   Thus, panning over to see the addon options, will disallow user to pan back to the left. 

See screenshots

Repro:
1) install android fennec nightly: Mozilla/5.0 (Android; Linux armv71; rv:2.0b12pre) Gecko/20110206 Firefox/4.0b12pre Fennec/4.0b5pre	
2) install addon with a long description (eg. Persona Theme: http://www.getpersonas.com/en-US/persona/356961)
3) Once installed, pan to Addons Manager and expand the description
4) Verify the description doesnt fit within the window alignment
5) Now pan over to see the full description and get to the addon buttons.  Verify unable to get back to the original view unless you hit the system back button

Expected:
- addon long description word wraps within alignment, and wont break panning

ActuaL:
- addon long descriptions break pannig

Comment 1

[Tony Chung [:tchung]]

Attachment: Addon long description screenshot

Comment 2

[Brad Lassey [:blassey] (use needinfo?)]

Tony can you elaborate on how this breaks panning? The visual bustage shouldn't block, but we don't quite understand the panning bustage.

Comment 3

[Tony Chung [:tchung]]

yah sure.  basically if you pan all the way to the right of the screen (to get screenshot 1), you are unable to pan the screen back to the left.   The workaround is to hit the system back button.

Let me add that i also had personas 0.8 installed with themes enabled in the original report.   I tried reproducing this on today's 2/8 nightly build, and i cannot reproduce the issue with that addon.  However, personas 0.8 also seems busted so im wondering if thats related.

Comment 4

[Tony Chung [:tchung]]

nevermind i reproduced it.   try this

STR
1) click to expand the long description addon
2) pan to the left about halfway or so
3) now click an adjacent addon that has a shorter description to expand that one
4) page shifts to my first screenshot, and unable to pan back over.

Comment 5

[Mark Finkle (:mfinkle) (use needinfo?)]

It looks like the description has no whitespace

Comment 6

[Tony Chung [:tchung]]

removing polish keyword.  it becomes a functionality bug due to the long description

Comment 7

[Brad Lassey [:blassey] (use needinfo?)]

Tony, in that case this is two different bugs, please file them seperately and then we'll deal with triaging them.

Comment 8

[Tony Chung [:tchung]]

per comment 7's suggestion, making this bug polish, removing blocking flag, edit title.   

bug 634377 now tracks the panning functionality issue.

Comment 9

[Wesley Johnston (:wesj)]

Fixed by Bug 634377

Comment 10

[Andreea Pod]

VERIFIED FIXED on:
Build ID: Mozilla /5.0 (Android;Linux armv7l;rv:2.0b13pre) Gecko/20110316
Firefox/4.0b13pre Fennec /4.0b6pre
Device: HTC Desire