Closed
Bug 509094
Opened 15 years ago
Closed 15 years ago
Copying bookmarks to clipboard (HTML format) should separate the links by newlines
Categories
(SeaMonkey :: Bookmarks & History, defect)
SeaMonkey
Bookmarks & History
Tracking
(Not tracked)
RESOLVED
FIXED
seamonkey2.0b2
People
(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)
References
Details
Attachments
(1 file)
|
1.32 KB,
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #509029 +++
Copying bookmarks creates both plain text and HTML formats on the clipboard. Bug 509029 fixes plain text to not end with a newline. For HTML the case is different, clients such as Composer or Word don't show trailing newlines.
Firefox 3.5 is separating bookmarks by \r\n\r\n and a final \r\n (at least on Windows). The empty line seems a bit excessive since whitespace is combined in HTML. I'd say just always add a single line break after each bookmark for copy as HTML.
|
Assignee | |
Comment 1•15 years ago
|
||
It appears that Composer, at least the SM 2 version of it, does show a trailing newline. Thus I lined the HTML case up with the plain text one, removing the trailing newline.
|
||
Updated•15 years ago
|
Attachment #393452 -
Flags: review?(neil) → review+
|
|
Assignee | |
Updated•15 years ago
|
Keywords: checkin-needed
|
||
Comment 2•15 years ago
|
||
Comment on attachment 393452 [details] [diff] [review]
proposed patch
[Checkin: Comment 2]
http://hg.mozilla.org/comm-central/rev/4e58ebdaff13
Attachment #393452 -
Attachment description: proposed patch → proposed patch
[Checkin: Comment 2]
|
|
||
Updated•15 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.0b2
You need to log in
before you can comment on or make changes to this bug.
Description
•