Closed Bug 313101 Opened 19 years ago Closed 18 years ago

HTTP auth dialog is titled "Prompt"

Categories

(Core :: Networking: HTTP, defect)

defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla1.8.1beta1

People

(Reporter: jruderman, Assigned: darin.moz)

References

(Blocks 1 open bug, )

Details

(Keywords: fixed1.8.1, polish)

Attachments

(1 file, 1 obsolete file)

Steps to reproduce: 1. Try to load http://www.squarefree.com/stats/ Result: HTTP Auth dialog comes up. Its title is "Prompt". Expected: A title like "Firefox" or "Password required". (Beltzner?)
Happens on Mac too (with Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20051019 Firefox/1.6a1). On Mac, the title is shown in bold in the body of the sheet.
OS: Windows XP → All
Hardware: PC → All
Originally reported by Burning Edge reader Moktoipas as a comment on http://www.squarefree.com/burningedge/2005/10/19/2005-10-19-branch-builds/.
Note that this is the same as in 1.0.7, so in answer to Matthias' question on the Burning Edge, "yes, this seems to be normal." That doesn't make it right, of course. "Password Requested" sounds right to me. :)
well, the dialog hasn't had a title at least since 1.2 <darin@netscape.com> 2001-05-11 14:03 (I didn't feel like trying to find the file where this code was in earlier...)
Attached patch patch? (obsolete) — Splinter Review
This changes the default title for PromptUsernameAndPassword and PromptPassword, for Firefox and Seamonkey. I guess this might be better if this had an explicit title instead of falling back to the default (which is what I assume the dependency on bug 265780 is about), but I don't think that should prevent us from using a better default, right?
*** Bug 333522 has been marked as a duplicate of this bug. ***
Flags: blocking1.8.1+
Keywords: polish
Target Milestone: --- → mozilla1.8.1beta1
Attachment #215600 - Flags: superreview?(darin)
Attachment #215600 - Flags: review?(mconnor)
Attachment #215600 - Flags: review?(neil)
Comment on attachment 215600 [details] [diff] [review] patch? So, this will probably seem insane, but for localizers to be aware of the change, we have to change the property name, otherwise they don't know we've updated the string in en-US and they need to retranslate...
Attachment #215600 - Flags: superreview?(darin)
Attachment #215600 - Flags: review?(neil)
Attachment #215600 - Flags: review?(mconnor)
Attachment #215600 - Flags: review-
Are you serious? That message has not been widespread enough. I see changes to strings happening all the time. Are you sure that's the best solution? Why don't they just check bonsai logs?
As far as I know, they mostly rely on the result of compare-locales, which is run on the l10n tinderboxen. It only shows failure if keys are added or removed (or renamed). I thought the rule was therefore that the requirement was to change the key if it's semantics change, not every time it's value changes.
Though I notice that most other locales have just translated the en-US string directly. Changing the key will let them know to make the same improvement we're making.
I don't know about you but it prompts "Password" when I try that URL.
Attached patch v1.1 patchSplinter Review
Attachment #215600 - Attachment is obsolete: true
Attachment #225443 - Flags: review?
Attachment #225443 - Flags: review? → review?(mconnor)
Attachment #225443 - Flags: approval-branch-1.8.1?(mconnor)
Attachment #225443 - Flags: review?(mconnor)
Attachment #225443 - Flags: review+
Attachment #225443 - Flags: approval-branch-1.8.1?(mconnor)
Attachment #225443 - Flags: approval-branch-1.8.1+
fixed-on-trunk, fixed1.8.1
Status: NEW → RESOLVED
Closed: 18 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: