Ticket #175 (closed enhancement: fixed)

Opened 5 years ago

Last modified 5 years ago

Use <PRE> element in storeArea to avoid character escaping issues

Reported by: BobMcElrath Owned by: MartinBudden
Priority: major Milestone: 2.2
Component: core Version:
Severity: medium Keywords:
Cc:

Description

Justin Case notes that adding a <pre> inside the <div> used to store tiddlers in the storearea fixes the problem with whitespace on IE. But we can go one step further and also remove the \n and \ escaping if we do this.

This has the added benefits of making a TW readable to text-only browsers, javascript-disabled browsers, and search engines.

The IE error is that upon saving, IE compresses a sequence of multiple spaces into one space.

Attachments

ticket175.patch Download (0.6 KB) - added by MartinBudden 5 years ago.

Change History

Changed 5 years ago by BobMcElrath

  • priority changed from undefined to minor
  • severity changed from undefined to medium
  • milestone set to 2.2

Changed 5 years ago by UdoBorkowski

Changed 5 years ago by JeremyRuston

  • summary changed from Remove escaping in the store to Use <PRE> element in storeArea to avoid character escaping issues

Changed 5 years ago by MartinBudden

  • owner changed from JeremyRuston to MartinBudden

Changed 5 years ago by MartinBudden

IE collapses multiple instances of a white space character into a single instance when loading <div>s. (Note this happens on loading, not saving.)

(Split out of ticket #8.)

Changed 5 years ago by MartinBudden

  • priority changed from minor to major
  • status changed from new to assigned

Changed 5 years ago by JeremyRuston

Implemented in changeset:1040

Changed 5 years ago by MartinBudden

Changed 5 years ago by JeremyRuston

  • status changed from assigned to closed
  • resolution set to fixed

Fixed in changeset:1811

Note: See TracTickets for help on using tickets.