Closed
Bug 247227
Opened 20 years ago
Closed 16 years ago
Cannot create shortcut for pages with no title or invalid characters in title
Categories
(Firefox :: Shell Integration, defect, P4)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: bmo, Unassigned)
References
Details
Attachments
(1 file, 1 obsolete file)
12.16 KB,
patch
|
asa
:
approval-aviary-
|
Details | Diff | Splinter Review |
This is the FireFox version of the following Mozilla bugs: bug 103468 = Mozilla creates .url files with zero byte at end, confusing IE (Internet shortcut) bug 78509 = Can't make an Internet shortcut with :, /, \ in title (by dragging link to desktop) bug 104933 = Mozilla-generated .url filename without name when page has no title These have all been fixed on the Mozilla 1.8 trunk. It would be worthwhile putting it into Firefox 1.0. The patch for all of these problems is patch 5 at bug 103468 http://bugzilla.mozilla.org/attachment.cgi?id=149548&action=view Note that this patch uses the localized string "noPageTitle" from chrome://navigator/locale/pageInfo.properties It is possible that this is different in Firefox?
This is a Windows only bug (in a Windows specific source file). The patch as applied to 1.8 trunk will fix Windows only, so I think you need to open a new bug for the same issue against Mac. Is there already a bug for the same problem on Seamonkey?
Comment 3•20 years ago
|
||
Bug 159773 is dup of this and l10n problem. Shift-JIS contains 0x5c(\). For Japanese this bug is bigger problem than for others.
Nominating for blocking 1.0 Patch available.
Flags: blocking1.0?
Updated•20 years ago
|
Flags: blocking1.0? → blocking1.0+
Priority: -- → P4
Comment 5•20 years ago
|
||
p4 priority - not a blocker. if a patch materializes, please nominate for aviary approval.
Flags: blocking-aviary1.0+ → blocking-aviary1.0-
For crying out loud read the bug, even just a little. There is a patch already. It's been in the trunk for ages. From comment #0: -- The patch for all of these problems is patch 5 at bug 103468 http://bugzilla.mozilla.org/attachment.cgi?id=149548&action=view --
I don't have access to source at the moment, nor am I doing development at moment, so I can't verify this patch still works. It's probably bitrot, as is normal with Moz development.
Comment 8•20 years ago
|
||
s/patch/fully reviewed patch/
This is the patch from bug 103468 simply applied and re-created for the the Firefox 1.0 source branch. It fixes the mismatch between the patch for the 1.8 trunk and the Firefox branch and now applies cleanly to the branch. Let's get this into the next Firefox point release to fix this problem. For examples of what this fixes see the testcases in attachment 149729 [details], 'testcases' on bug 103468.
Attachment #161099 -
Attachment is obsolete: true
Reporter | ||
Comment 10•20 years ago
|
||
Comment on attachment 166229 [details] [diff] [review] fully reviewed patch updated for firefox 1.0 branch This patch is unchanged from that which has R/SR at bug 103468.
Attachment #166229 -
Flags: approval-aviary?
Comment 11•20 years ago
|
||
The plan at this point is to push aggressively towards 1.1, with 1.0.x being security-only releases, so I doubt we'll take this on branch.
Updated•20 years ago
|
Attachment #166229 -
Flags: approval-aviary? → approval-aviary-
Comment 12•20 years ago
|
||
*** Bug 281982 has been marked as a duplicate of this bug. ***
Updated•17 years ago
|
Assignee: bugs → nobody
Comment 13•16 years ago
|
||
Firefox 1.0 is no longer supported so resolving -> wontfix Note: this was fixed on trunk quite some time ago
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → WONTFIX
Version: unspecified → 1.0 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•