Ticket #13566 (confirmed Bug)

Opened 3 years ago

Last modified 2 years ago

TinyMCE filters start attribute in lists

Reported by: poetzsch Owned by:
Priority: minor Milestone: 4.x
Component: Visual Editor Version: 4.3
Keywords: TinyMCE Cc: keul

Description

The start attribute is not deprecated in HTML5: "The value attribute of the li element is no longer deprecated as it is not presentational. The same goes for the start and type attributes of the ol element."  http://www.w3.org/TR/html5-diff/

But in Plone 4.2.4 and 4.3 TinyMCE is filtering this attribute.

Change History

comment:1 Changed 3 years ago by kleist

  • Status changed from new to confirmed
  • Component changed from Unknown to Visual Editor

comment:2 Changed 2 years ago by keul

  • Cc keul added

Fix for Plone 4.2/TinyMCE 1.2 is in this pull request:  https://github.com/plone/Products.TinyMCE/pull/70

Speaking of Plone 4.2, apart the fix itself, there are some public.css issues to be fixed. Those styles will hide the new attribute effects:

#content ol, dl.portlet ol, div.portletStaticText ol {
    list-style-type: decimal;
}
* ul, ol {
    list-style: none outside none;
}

I've no idea the target of those rules,. I tried to fix with some additional rules, but it's not working:

#content ol[type],
dl.portlet ol[type],
div.portletStaticText ol[type] {
    list-style-type: inherit;
}
ol[type] {
    list-style: inherit;
}

Now, let's check this on Plone 4.3.

Last edited 2 years ago by keul (previous) (diff)

comment:3 Changed 2 years ago by keul

This is the pull request for PLone 4.3/Tiny 1.3.x:  https://github.com/plone/Products.TinyMCE/pull/71

Same CSS problems as above.

comment:4 Changed 2 years ago by keul

... PS: if this will be accepted, even if no future Plone 4.2 release are planned, a release of TinyMCE 1.2 will be really appreciated! :)

comment:5 Changed 2 years ago by davisagli

I merged the pull requests.

Note: See TracTickets for help on using tickets.