New markup for eliding content in Wikka editor

[BrianKoontz]

It would be useful to be able to create comments in Wikka markup or otherwise preclude certain content from being rendered on a page. Two new markup tags are introduced:

/* Content to be elided */

This tag prevents the enclosed text from being rendered to the displayed page. It also eliminates trailing whitespace so there are no gaps in the output.

``Content to be elided``

This tag is similar to the previous tag, except trailing whitespace is preserved.

[BrianKoontz]

(In [1263]) Introduce new markup to elide content in Wikka markup from appearing on displayed page. Cleaned up FormattingRules and added new markup tags, as well as existing tags for insertion and deletion. Refs #830.

[DarTar]

Brian, nice and useful addition. Two issues:

• what is the rationale for introducing two different kinds of markup with or without trailing whitespace?
• ``this markup`` breaks inside code blocks (backticks are common in programming languages so this may not be a good markup option.

[BrianKoontz]

When I first coded this, I noticed that not dealing with the trailing whitespace (actually line terminators) left ugly spacing...I couldn't decide whether everyone would be happy with removing the trailing whitespace, so I just created another tag to leave the whitespace there (I'm sure someone might have a reason to do that!).

Indeed, ... does break inside the %%...%% tags. Strange, since /*...*/ is unaffected. Well, I can either toss out the tag, or fix it so it works inside a code block.

[DarTar]

Retargeting to 1.3. Code for this ticket has already been committed to trunk, from which 1.3 will be branched. Consider backporting urgent issues to 1.2.X

[DarTar]

I still think we should go for a single markup flavour (my preference goes to /* */) with whitespace trimmed.

[DotMG]

this markup breaks inside code blocks

[DotMG]

I meant backticks inside code blocks make the whole content of the codeblock to be discarded.

[DotMG]

(In [1641]) refs #830

For code blocks (%% ... %%) containing backticks comments ( ... ), the whole content of the code block is discarded. Added ^{and $ anchors to the preg patter for backticks.}

[BrianKoontz]

Tested with embedded ... and /*...*/ lines in %%...%% markup. Nothing was deleted in the rendered page, as is expected.

[BrianKoontz]

(In [1668]) Fixed comment style radio buttons so that they display the correct user settings; modified labels to reflect user-slected comment styles; changed default comment style to threaded (can be overridden by user); removed unused commentsnav div in code and CSS. Refs #830.

[BrianKoontz]

Replying to BrianKoontz:

(In [1668]) Fixed comment style radio buttons so that they display the correct user settings; modified labels to reflect user-slected comment styles; changed default comment style to threaded (can be overridden by user); removed unused commentsnav div in code and CSS. Refs #830.

This change referenced the wrong ticket. The correct ticket is #380.

[BrianKoontz]

Changed milestone to 1.3.1