Closed Bug 341278 Opened 18 years ago Closed 18 years ago

Status bar priority

Categories

(Camino Graveyard :: Toolbars & Menus, defect)

PowerPC
macOS
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: acoolie, Assigned: stuart.morgan+bugzilla)

Details

(Keywords: fixed1.8.1.1)

Attachments

(1 file, 1 obsolete file)

User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.0.3) Gecko/20060427 Camino/1.0.1 Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.0.3) Gecko/20060427 Camino/1.0.1 When loading a page, Camino shows things such as "Connecting to..." and others. These messages are helpful usually, but when trying to find out where a link goes (A feature normally implemented in Camino), it is quite hard to see. The instant a new connection is made, the href of the link goes away. Basically, I feel that link location should have priority over most other messages in the status bar. Reproducible: Always Steps to Reproduce: 1. Go to google.com (Or maybe a page that takes longer to load) 2. Quickly hover over a link Actual Results: The link location quickly disappears. Expected Results: The link location should stay in place until mouseout.
Confirming. This should be reasonable to do, since it looks like the link information is coming in through setStatus:OfType, whereas the various loading strings are coming from onLoadingStarted/onLoadingCompleted/onStatusChange:. Taking.
Assignee: nobody → stuart.morgan
Severity: normal → minor
Status: UNCONFIRMED → NEW
Ever confirmed: true
Attached patch prioritize status strings (obsolete) (deleted) — Splinter Review
fix
Attachment #243442 - Flags: review?
>+ eStatusLinkTarget = 0, // like mouseover info That should read 'link mouseover info'; I'll fix that on respin after it's reviewed.
Comment on attachment 243442 [details] [diff] [review] prioritize status strings r=cl. Looks good. >- mTabTitle = [mLoadingStatusString retain]; >+ mTabTitle = [NSLocalizedString(@"TabLoading", @"") retain]; This doesn't require any string changes, does it? cl
Attachment #243442 - Flags: review? → review+
Comment on attachment 243442 [details] [diff] [review] prioritize status strings Nope, same string we've always used.
Attachment #243442 - Flags: superreview?(mikepinkerton)
Attachment #243442 - Flags: superreview?(mikepinkerton) → superreview?(joshmoz)
Comment on attachment 243442 [details] [diff] [review] prioritize status strings sr=pink in the .h, |mStatusStrings| should have a comment that it's a strong (owning) reference.
Attachment #243442 - Flags: superreview?(joshmoz) → superreview+
Attached patch with comments (deleted) — Splinter Review
As checked into trunk and MOZILLA_1_8_BRANCH
Attachment #243442 - Attachment is obsolete: true
Status: NEW → RESOLVED
Closed: 18 years ago
Keywords: fixed1.8.1.1
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: