453409 - Calendar list scrollbar hides read-only icons

Status: VERIFIED FIXED

(Calendar :: General, defect)

Description

[Daniel Boelzle [:dbo]]

follow-up from bug 392561:
In case there's not enough vertical space to show up the whole calendar list, the scrollbar hides the read-only icons.

Comment 1

[Philipp Kewisch [:Fallen][📱📆]]

Attachment: Work around tree problems - v1

After lots of ugly workarounds, I have found something that is semi-clean. Enjoy!

Comment 2

[Berend Cornelius]

> +            <!-- This is a hack to make sure the last column is not covered by
> +                the scrollbar in case of an overflow.

I think that a "hack" is a mistakable description of your patch. I would rather call it a "very elegant workaround". There's no need to hide your light under a bushel!
patch works great; r=berend

Comment 3

[Philipp Kewisch [:Fallen][📱📆]]

(In reply to comment #2)
> I think that a "hack" is a mistakable description of your patch. I would rather
> call it a "very elegant workaround". There's no need to hide your light under a
> bushel!
I've adapted the comment to contain the new commendations ;-)

Checked in on HEAD and MOZILLA_1_8_BRANCH

-> FIXED

Comment 4

[Andreas Treumann]

Checked in lightning 2008090403 and sunbird 20080903 -> VERIFIED