699312 - Selected tab should be bold in the tabs-list

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Sriram Ramasubramanian [:sriram]]

The selected tab should be shown in bold in the list of tabs shown.

Comment 1

[Sriram Ramasubramanian [:sriram]]

Attachment: Patch

This patch makes the selected tab bold in the list.

I tried make the title bold when the view is created. However, it had weird issues that I couldn't understand. The first item in the list is always bold along with the selected tab in the list. Probably some race conditions, I believe.

I tried using onWindowFocusChanged in activity -- as this is when the list gets populated -- however, this doesn't help us when the tabs gets updated in the background. The background changes of url/title, causes the list to render again, and we don't know if that has happened yet. thisList.setAdpater() doesn't ensure that views are created immediately after this call.

The last option was to use a Handler to update the views after a timeout. I've used a 10ms timeout and this works fine with my slow Nexus S.

There is a method addOnLayoutChangeListener() with ListView, but this is available only from API Level 11.

I would be happy to change the logic, if we can find a better solution by adding bold when view is created itself.

Comment 2

[Sriram Ramasubramanian [:sriram]]

Attachment: Patch

The previous patch had a wrong bug number in the message.

Comment 3

[Sriram Ramasubramanian [:sriram]]

Attachment: Patch

This works fine now :)
I forgot the "resusability" nature of convertView, which caused all issues initially.
Now, the views are not reused (and there is no need to reuse smaller views with text and image) and the fix is even simpler.
There is no flickering while tab updates in the background.

Comment 4

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

https://hg.mozilla.org/projects/birch/rev/15595d5edb2c

Comment 5

[Aaron Train [:aaronmt]]

Mozilla/5.0 (Android; Linux armv7l; rv:10.0a1) Gecko/20111003 Firefox/10.0a1 Fennec/10.0a1
Samsung Nexus S (Android 2.3.6)