copy-paste-format-internal --Copied text from Sophie loses format when pasted

[deyan]

1. Create 2 text frames in Sophie and type some text in both of them.
2. Change the style of the text in one of the frames, copy it, and paste it in the other frame.

The formatting is lost.

[vlado]

-> Copy/Paste of formatted text is not supported for now (only plain text).
-> The pasted text is appended after the existing one in the frame.

[vlado]

Bug fixed on branch branches/private/vlado/demo1-bugfix in [3918].

[meddle]

If we consider this as a quick fix for the demo, it works, but that thing just kills one of the optimizations you made returning the pasting to the old logic, character by character, other thing when we paste and the cursor is not in the end of the current text, let's say in the middle of it, the pasted text is appended to the end of the text and not when the cursor is. May be later this bug have to be reopened.

Merged to the trunk in [3937].

3p 20m

[dido]

Not fixed or not committed.

[tanya]

[5170]

[tanya]

[5175]

[tanya]

[5182]

[meddle]

So... because of that that the styling comes with the first letter, and not every time is copied we will fail it.

[boyan]

Now the existing text is preserved but still the formatting is lost. Since the task is too old, moving it to analysis_finished

[boyan]

Changing description to be actual.

[deyan]

Batch update from file report_1.csv

[deni]

• The main idea is that ImmHotTextTransferable should carry HotTextData to paste text inside Sophie.
  • Add HotTextData in ImmHotTextTransferable.registerData(List<DndData> dataList).
  • Process HotTextData in ON_PASTE operation in HotTextLogic.

• HotTextData will still be carried in the form of RawBinData. Converting ImmHotText to OutputStream will be done in two phases:
  • persisting ImmHotText to Storage (using MasterPersister);
  • creating an XML Document and writing it in the stream (using some helper methods of ZipPackagePersister;

• Move some logic from ZipPackagePersister to public static helper methods:
  • public static void unpackXml(InputStream stream, Storage target) throws IOException - reads from a given InputStream and unpacks the XML data into Storage.
  • public static void writeXmlToStream(Document xml, OutputStream stream) throws IOException - writes the given XML Document in the given OutputStream.

• In ImmHotText.replace(HotInterval interval, ImmHotText insertText) make sure not to change the ID of the text when applying styles.

• Source code: [8183], [8199]

[deni]

• changed ImmHotText.getNormalizedStyleValues() to respect style values that begin before or end after the end of a subtext in [8201].
• removed an anonimous class in [8203].

[pap]

• Seems Ok.
• I am glad you wrote tests or them methods you changed. I hope you did that before the actual change :)
• It would be good if you fix bad code by your own initiative (I mean variable names, spacing and such things).
• Good luck.

[deni]

• fixed drag-out of AllResourcesPalette to copy styled text, too and removed the unnecessary linefeeds at the end of rtf data in RTFTextImportManager in [8208]

[deyan]

Batch update from file 0911261.csv

[pap]

• Commited to trunk in [8223].
• Left commented out code in HotTextData.
• The paste(EventR3, DndTransferable) method would better be ImmHotText getText(DndTransferable).
• It would be better if you call the getText in pasteText(EventR3, ImmHotText) and make it use a transferrable and I hope you get the idea.
• I don't like giving the EventR3 as a parameter. It is much better to require a TextView.
• I don't understand why did you make a second unpackXml method in ZipPackagePersister. You could've changed it to have InputStream as an argument and change the 2 places where it is called.
• Attachments(links) are not transfered between frames.

[deyan]

links should be fixed