Closed
Bug 276819
Opened 20 years ago
Closed 19 years ago
drag'n'drop URLs to tab strip should ignore trailing spaces
Categories
(SeaMonkey :: UI Design, enhancement)
Tracking
(Not tracked)
VERIFIED
DUPLICATE
of bug 138245
People
(Reporter: csthomas, Unassigned)
References
()
Details
(Keywords: helpwanted)
Attachments
(1 obsolete file)
A trailing space should be ignored in text dragged to the tab strip. A leading space seems to be ignored already. Testcase: data:text/html,text http://www.mozilla.org text
| Reporter | ||
Comment 1•20 years ago
|
||
This strips a single trailing space.
Attachment #170152 -
Flags: superreview?(neil.parkwaycc.co.uk)
Attachment #170152 -
Flags: review?(timeless)
| Reporter | ||
Updated•20 years ago
|
Status: NEW → ASSIGNED
Whiteboard: [cst:active]
Comment 2•20 years ago
|
||
Comment on attachment 170152 [details] [diff] [review] patch Odd that drag'n'drop trims leading spaces, bug perhaps? Anyway, this isn't the preferred way to trim spaces. You haven't even checked if url is null.
Attachment #170152 -
Flags: superreview?(neil.parkwaycc.co.uk) → superreview-
| Reporter | ||
Updated•20 years ago
|
Attachment #170152 -
Attachment is obsolete: true
Attachment #170152 -
Flags: review?(timeless)
Comment 3•20 years ago
|
||
There exists a preference for the dropping of newlines, so I can cut&paste a multiline URL to the Location Bar. Would be fine if this could also be included here, so dragging acts like pasting. Bug 50935 Multiline paste in text field should be switchable see Bug 50935 comment 5 for definition of editor.singleLine.pasteNewlines
| Reporter | ||
Comment 4•20 years ago
|
||
Neil thinks it may be a bug in the serializer that currently strips the leading space.
| Reporter | ||
Updated•20 years ago
|
Assignee: cst → guifeatures
Comment 5•19 years ago
|
||
Please re-open if you have a reason to believe this is not a duplicate. *** This bug has been marked as a duplicate of 138245 ***
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → DUPLICATE
Updated•18 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•