Skip to content
This repository was archived by the owner on Oct 4, 2021. It is now read-only.

[SourceEditor2] Allow subclasses to modify loaded/saved text #646

Merged
merged 1 commit into from
Sep 18, 2014

Conversation

sandyarmstrong
Copy link
Member

Hypothetically, a subclass of SourceEditorView might want to store metadata
in a hidden header area of the file. Overriding ProcessLoadText gives the
subclass an opportunity to extract said metadata, and ensure that the text
loaded into the Document excludes it.

Similarly, ProcessSaveText would allow this subclass to inject updated
metadata into the file text just before the write occurs.

Hypothetically, a subclass of SourceEditorView might want to store metadata
in a hidden header area of the file. Overriding ProcessLoadText gives the
subclass an opportunity to extract said metadata, and ensure that the text
loaded into the Document excludes it.

Similarly, ProcessSaveText would allow this subclass to inject updated
metadata into the file text just before the write occurs.
@Therzok
Copy link
Contributor

Therzok commented Sep 17, 2014

/cc @mkrueger

@sandyarmstrong
Copy link
Member Author

For this branch, it's actually just depending on a 👍 from @abock.

sandyarmstrong added a commit that referenced this pull request Sep 18, 2014
[SourceEditor2] Allow subclasses to modify loaded/saved text
@sandyarmstrong sandyarmstrong merged commit 016f239 into mono:monodevelop-xpg Sep 18, 2014
@sandyarmstrong sandyarmstrong deleted the hooks2 branch September 18, 2014 02:14
@xamarin-release-manager
Copy link
Contributor

Fixed in version 8.5.0.577 (master)

Pull Request #8302 merged by: Vsevolod Kukol
Author: mono
Commit: 62e5edc (mono/monodevelop)

@xamarin-release-manager
Copy link
Contributor

Fixed in version 8.4.0.1901 (release-8.4)

Pull Request #8302 merged by: Vsevolod Kukol
Author: mono
Commit: 62e5edc (mono/monodevelop)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants