Closed
Bug 51025
Opened 25 years ago
Closed 25 years ago
":" should be removed from protocols in the Desktop Integration
Categories
(SeaMonkey :: Preferences, defect, P3)
Tracking
(Not tracked)
VERIFIED
WONTFIX
People
(Reporter: bugzilla, Assigned: bugzilla)
Details
(Keywords: polish)
Attachments
(1 file)
|
1.46 KB,
patch
|
Details | Diff | Splinter Review |
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"
| Reporter | ||
Comment 2•25 years ago
|
||
| Reporter | ||
Updated•25 years ago
|
| Reporter | ||
Comment 4•25 years ago
|
||
patch is saying: check me in...
| Assignee | ||
Comment 5•25 years ago
|
||
r=me, cc brendan for checkin approval
Comment 6•25 years ago
|
||
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•25 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•25 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.
Comment 9•25 years ago
|
||
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•25 years ago
|
||
marking wontfix
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → WONTFIX
Updated•21 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•