":" should be removed from protocols in the Desktop Integration

VERIFIED WONTFIX

Status

SeaMonkey
Preferences
P3
normal
VERIFIED WONTFIX
18 years ago
13 years ago

People

(Reporter: Henrik Gemal, Assigned: Blake Ross)

Tracking

({polish})

Trunk
x86
Windows 2000
polish

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

18 years ago
In the Desktop Integration it states different protocols. Why does it says 
"http:" and not just "http".

"http:" should be "http"
"https:" should be "https"
"ftp:" should be "ftp"
"chrome:" should be "chrome"
good point. should be trivial to fix for beta3.
Keywords: correctness, nsbeta3, polish
(Reporter)

Comment 2

18 years ago
Created attachment 13927 [details] [diff] [review]
Diff to remove ":"
(Reporter)

Updated

18 years ago
Keywords: patch, review
(Reporter)

Comment 3

18 years ago
here you go...
Assignee: matt → BlakeR1234
(Reporter)

Comment 4

18 years ago
patch is saying: check me in...
(Assignee)

Comment 5

18 years ago
r=me, cc brendan for checkin approval
Status: NEW → ASSIGNED
Keywords: review → approval
Maybe it's trying to teach people how to distinguish URL schemes?  If you take
out the colon, then it looks like you're using lower-case acronyms, in which
case why aren't they upper per all English style manuals (HTTP, HTTPS, etc.)? 
Is it really clear without the : that these are URI scheme names?

/be
(Assignee)

Comment 7

18 years ago
I also wasn't sure if removing the colons is a good idea.  It just seems like a 
step down as far as the user understanding it goes.  cc'ing verah for final 
word.

Comment 8

18 years ago
I think that the colons were there to give users a clue about where they might 
see these items. So they do serve a purpose in this context. I don't think it's 
of earth-shattering importance, but IMHO the colons should stay.
I agree with verah, and can't approve the patch.  I'd say this bug is INVALID,
but I'll leave it to the assignee to so resolve it.

/be
(Assignee)

Comment 10

18 years ago
marking wontfix
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → WONTFIX
okay. vrfy.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.