Desktop: Fixes #9304: Fix HTML resource links lost when editing notes in the rich text editor (Backport #9435)

[personalizedrefrigerator]

Summary

Backports #9435 into release-2.13.

Rationale

• Even though this doesn't fix a regression, Editing an HTML note in the rich text editor removes resource links #9304 combined with the issue fixed by Desktop: Fixes #9613: Fix rich text editor deletes HTML links to notes #9624 is causing data loss for some users.

[laurent22]

There's a linter failure on one of the file, although you haven't modified that file. But could you check what might be the issue? I thought it was due to my own change but I see the extra space it's complaining about is not one my own file or even on dev branch, so this is strange

[personalizedrefrigerator]

is not one my own file or even on dev branch

It seems that the relevant file was last changed in this commit: 229b9a5

There do seem to be trailing spaces tabs in that commit (as highlighted below):

Relevant content as text

	// Disabled for now because the ENEX parser has become so error-tolerant
	// that it's no longer possible to generate a note that would generate a
	// failure.
	
	// it('should keep importing notes when one of them is corrupted', async () => {
	// 	const filePath = `${enexSampleBaseDir}/ImportTestCorrupt.enex`;
	// 	const errors: any[] = [];
	// 	const consoleSpy = jest.spyOn(console, 'warn').mockImplementation(jest.fn());
	// 	await importEnex('', filePath, {
	// 		onError: (error: any) => errors.push(error),
	// 	});
	// 	consoleSpy.mockRestore();
	// 	const notes:NoteEntity[] = await Note.all();
	// 	expect(notes.length).toBe(2);
	// 	expect(notes.find(n => n.title === 'Note 1')).toBeTruthy();
	// 	expect(notes.find(n => n.title === 'Note 3')).toBeTruthy();

I don't see test output in CI for that commit, so perhaps the linter didn't run in CI either?

[laurent22]

It seems that the relevant file was last changed in this commit: 229b9a5

There do seem to be trailing spaces tabs in that commit (as highlighted below):

For some reason, I'm not seeing it in my environment and running yarn linter ./ also doesn't result in any change. If you have this extra tab in your env, would you mind removing it and adding the fix to this pull request please?

[laurent22]

There's just one conflict to fix and we're good to go