Closed Bug 167814 Opened 22 years ago Closed 20 years ago

Subscribe's progress meter extends past the tree's content outline.

Categories

(SeaMonkey :: MailNews: Message Display, defect)

defect
Not set
minor

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: stephend, Assigned: Stefan.Borggraefe)

Details

Attachments

(3 files)

Build ID: 2002-09-10-08, All OS, Modern and Classic.

Summary: Subscribe's progress meter overextends past the tree's content outline.

Steps to Reproduce:

1.  Launch Subscribe
2.  Click Refresh
3.  Look at the progress meter's overlapping with the subscribe tree's outline.

Expected Results:

It should cut off right where the edge of the outliner ends.

Actual Results:

It extends slightly past.  Note that this happens on Classic and Modern, so I'm
pretty sure it's not theme specific.
Summary: Subscribe's progress meter overextends past the tree's content outline. → Subscribe's progress meter extends past the tree's content outline.
Attached patch FixSplinter Review
This patch gives the status bar the same margin as the tree above it. Tested
with Linux and Mac OS X (both Classic and Modern).

Thunderbird has this already fixed in the Pinstripe theme on Max OS X. I assume
once Qute is replaced by xyz-stripe this will be fixed on the other platforms,
too. In any case I don't think it makes sense to fix the Qute theme anymore.
Please correct me if I'm wrong.
Assignee: sspitzer → Stefan.Borggraefe
Status: NEW → ASSIGNED
Attachment #154100 - Flags: review?(neil.parkwaycc.co.uk)
Comment on attachment 154100 [details] [diff] [review]
Fix

This sorta works on modern, because of the 4px statusbarpanel padding... I
suppose it's okay.
Attachment #154100 - Flags: review?(neil.parkwaycc.co.uk) → review+
Comment on attachment 154100 [details] [diff] [review]
Fix

Thanks for the review, Neil!

Seth, can you sr this patch?
Attachment #154100 - Flags: superreview?(sspitzer)
Comment on attachment 154100 [details] [diff] [review]
Fix

sr=sspitzer
Attachment #154100 - Flags: superreview?(sspitzer) → superreview+
you might want to look at how mscott (?) fixed this issue in tbird.
(In reply to comment #6)
> Created an attachment (id=154205)
> screen shot of subscribe dialog from tbird (aviary 1.0 branch)
> 
> you might want to look at how mscott (?) fixed this issue in tbird.

When you look at the screenshot closely, you see that this isn't fixed in the
Qute theme. The right border of the status bar is to far right. This is more
obvious with a Linux Gtk2 build, because there you get a different border
(probably the same is true when you turn off Windows XP's Luna theme, but I
can't test this at the moment).

But since this *is* fixed in the Pinstripe theme on the Mac, I assume this will
be fixed on the other platforms too when they are switched from Qute to *stripe.

You can see a screenshot of this area of the Subscribe dialog in a Mac OS X TB
build here: http://www.kuestner.org/images/stefan_b/TB-Pinstripe.pdf

I'll keep an eye on this and file/fix a TB-specific bug in case this isn't fixed
after the switch to *stripe on the other platforms.

I checked in the patch for Modern/Classic. Marking FIXED.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Verified FIXED (at least for my taste) on Windows XP Seamonkey, with Classic and
Modern themes, using build 2004-07-28-08
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
Component: MailNews: Subscribe → MailNews: Message Display
QA Contact: stephend → search
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: