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)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED
seamonkey2.0b2

People

(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)

References

Details

Attachments

(1 file)

+++ 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.
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.
Assignee: nobody → jh
Status: NEW → ASSIGNED
Attachment #393452 - Flags: review?(neil)
Attachment #393452 - Flags: review?(neil) → review+
Keywords: checkin-needed
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]
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.