In navigator it is possible to move the mouse cursor all the way to the right, press the mouse button, and scroll. In the message pane, you have to move the cursor a bit to the left because there is a border on the right side of the scrollbar. This is annoying.

This is similar to the recently fixed bug 52888, "classic skin: scrollbar should be at right edge of window" (for navigator).

Yo! Could someone please review the patch?

Updating multiple bugs. This is a valid UI issue. Would be nice to have it fixed if time allows.

shuehan / joe, could you review and then I'll sr?

ah, I get it now. if I expand my 3 pane or my stand alone msg window to take up the whole monitor, I can't move my mouse to the edge of the monitor and click and scroll the message pane. but I can do that once I apply the patch, at least for modern. I'm not sure the classic changes are correct, or needed. #messagepane doesn't define a right-border in classic (mac or windows, don't forget about mozilla\themes\classic\messenger\mac\mailWindow1.css but the modern changes are fine. for the classic changes, can you double check if they are needed? cc andreww, so he can review the classic changes.

> I'm not sure the classic changes are correct, or needed. > #messagepane doesn't define a right-border in classic No, but #messagepanebox does, and that's what I removed it from. Before I did that, I couldn't scroll the message with classic when the mouse was at the right edge of the screen, now I can. But it was good that you told me to doublecheck it, because that made me discover that the thread pane has the same problem (in classic only). I guess that that should actually be filed as a separate bug, but we might as well fix it here.

Could someone review?

Comment on attachment 61870 [details] [diff] [review] patch - fixes the bug for both message and thread pane looks ok to me.

Comment on attachment 61870 [details] [diff] [review] patch - fixes the bug for both message and thread pane r=andreww

You're going to need to re-do this patch, the CSS changed quite a bit after my themes re-write last month.

Better?

my question would be did you test this new patch on your own build?

Tested it on on build 2001122803. Why, is there a problem with it?

andreww: ping!

Comment on attachment 63422 [details] [diff] [review] updated patch r=andreww - check with hewitt as well if you can.

hewitt, can you sr=?

Seth, could you sr= this, please?

adding to tracker for rotting patches...

Comment on attachment 63422 [details] [diff] [review] updated patch sr=sspitzer

Finally... thanks, Seth! Andreww, could you check this in for me, please?

ok - Ill do it when the tree opens again for general checkins...

Andrew? Did you plan to check this in after 1.0, or did you just forget about it?

I should be able to get this in today when the tree opens. Ill send an approval req right now.

thanks for the reminder :)

Comment on attachment 63422 [details] [diff] [review] updated patch a=asa (on behalf of drivers) for checkin to the 1.0 trunk

ok going to try to get this in today.

ok lets try again today. Several fire drills on friday kept me from doing this.

fix checked in.

Verifying. Thanks, Andrew! :-)

Bug 143855: a better fix would be to somehow show the border to the left of the scrollbar.