Feature Proposals » New auto mode for Twisty Plugin

Motivation

Better usability and less confusion when using the TwistyPlugin.

Description and Documentation

TWikibug:Item6242 was filed out of a confusion/usability issue: The default mode="span" breaks twisties when the twisty contains block elements, such as div tags, more than one paragraph, tables or bullets.

DontMakeMeThink proposal to fix the usability issue:

• Add a new mode="auto" that uses a span tag if content within twisty is not a block element, else uses the div tag.
• Make the mode="auto" the default mode, augmenting the existing mode="span" and mode="div".

There is a slight chance that some existing content will behave differently. However, currently if a twisty containing block elements is used without the div mode, the plugin generates invalid HTML.

Examples

Impact

Implementation

-- Contributors: PeterThoeny - 2011-12-15

Discussion

Accepted now based on 7 day feedback period rule.

-- PeterThoeny - 2012-01-02

I'm taking over implementation of this, as per discussion in DoNotUseSelfclosingPTag

-- Mahiro Ando - 2013-04-10

I tried implementing it, but soon I realized it would not be easy to get it done in a short period of time.

For now, I will work around our internal issue by setting "TWISTYMODE = div" as the firm-wide default, since the majority of the actual usage seems to be for the block mode while people just forget mode="div".

I have unset the CommittedDeveloper field, so anyone please feel free to take the ownership.

-- Mahiro Ando - 2013-04-15