Currently the text we can specify to be shown at the top of a new issue strips tags (at least <p>). I would really like to be able to separate paragraphs because there are several conditions that need to be separate for potential issue creators. Here's an example of three separate conditions all run together so that they lose meaning.

I wasn't sure whether this belonged here or in redesign.

Comments

nancydru’s picture

BTW I would be happy with using check_markup and letting it use DEFAULT_FILTER_FORMAT.

michelle’s picture

I'd like this, too. A solid wall of text is hard to read and even more likely to be ignored.

Michelle

greggles’s picture

Title: Don't strip tags from "Create issue" text. » Use default format for "Submission guidelines" text.
Project: Drupal.org site moderators » Project issue tracking
Version: » 6.x-1.x-dev
Component: Content moderation » Issues
Priority: Minor » Normal
Status: Active » Needs review
StatusFileSize
new772 bytes

I've been annoyed by this before as well. Currently it is filtered by "filter_xss" without a second argument which removes xss, but also removes any tags that are "block" elements (i.e. p, br, etc.).

The only drawback I know of switching from filter_xss to check_markup is that check_markup by default adds a p tag around the block of text which can add more whitespace than necessary in some circumstances.

nancydru’s picture

Thanks, Greggles. In this case, I would rather have more whitespace than a solid block of text. But if it is objectionable, then perhaps filter_xss_admin would work.

Since I want to see this on DO, how can it be reviewed and tested? The code looks fine to me.

greggles’s picture

Since this setting can be edited by a large number of users who may not be totally trustworthy I don't feel that filter_xss_admin is appropriate.

In terms of how to get it reviewed and tested...install project and project_issue and test it.

dww’s picture

Category: feature » task
Status: Needs review » Needs work
Issue tags: +needs drupal.org deployment

Good idea. Tested and it works fine. Only change is that we should do the same for the site-wide help text (which appears just above in the form). Committed that to both HEAD and DRUPAL-5--2.

Now this just need to be deployed on d.o. Luckily, we've got downtime in 2 hours. ;)

nancydru’s picture

Fabulous, thanks to both of you. And downtime means a chance for a quick nap...

dww’s picture

Status: Needs work » Fixed
Issue tags: -needs drupal.org deployment
nancydru’s picture

And being used successfully.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.