Ticket #198 (closed enhancement: wontfix)

Opened 2 years ago

Last modified 2 years ago

Xhtml parser strips ordered list attributes (type, start, compact)

Reported by: alex Owned by: jf.hovinne
Priority: major Milestone: 0.5
Component: editor Version: trunk
Keywords: Cc:

Description

summary says it all. I created a patch to fix it but am unsure if it is correct. please review.

Attachments

wymeditor-ol-attrs.patch Download (480 bytes) - added by alex 2 years ago.

Change History

Changed 2 years ago by alex

comment:1 Changed 2 years ago by mr_lundis

  • Status changed from new to closed
  • Type changed from defect to enhancement
  • Resolution set to wontfix

All these attributes are not supported when using a strict dtd, and the type and compact attributes are purely presentational. Therefore they wont be included as WYMeditor is a WYSIWYM (What You See Is What You Mean) editor. See the  WYMeditor Concepts page for an (a bit outdated) introduction.

The start attribute I could see some practical use for. But I wonder if the usage of this attribute is big enough to motivate inclusion, even though it's no longer included in the specifications.

Regarding your patch, I think it should work fine. But for now I'm setting this as wontfix due to the reasons I mentioned above.

However, I'm glad to see that you want to contribute to the WYMeditor project. Cheers! ;)

comment:2 Changed 2 years ago by alex

Cool - I thought IE6/7 needs the type="" attribute, but it seems like I was too tired yesterday ;)

Note: See TracTickets for help on using tickets.