Skip to:
Content

bbPress.org

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#2483 closed defect (bug) (fixed)

Allow target="" in links by default

Reported by: wpdennis's profile wpdennis Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.5.1 Priority: lowest
Severity: minor Version:
Component: General - Content Creation Keywords: has-patch
Cc:

Description

The editor is showing the checkbox "Open link in a new window/tab" in its add link popup, by default.

But using it doesn't work for participants, since target="" isn't allowed. It would be more consistent to allow it by default or to remove the checkbox from the add link popup.

Allowing it in formatting.php:

// Links
'a' => array(
		'href'     => array(),
		'title'    => array(),
		'rel'      => array(),
		'target'      => array()
),

Attachments (1)

2483.diff (401 bytes) - added by netweb 11 years ago.

Download all attachments as: .zip

Change History (6)

#1 @johnjamesjacoby
11 years ago

  • Milestone changed from Awaiting Review to 2.5.1

@netweb
11 years ago

#2 @netweb
11 years ago

  • Keywords has-patch added; dev-feedback removed

#3 @johnjamesjacoby
11 years ago

  • Owner set to johnjamesjacoby
  • Resolution set to fixed
  • Status changed from new to closed

In 5205:

Add target to anchor in bbp_kses_allowed_tags(). Props wpdennis. Fixes #2483. (trunk)

#4 @johnjamesjacoby
11 years ago

In 5206:

Add target to anchor in bbp_kses_allowed_tags(). Props wpdennis. Fixes #2483. (2.5)

#5 @johnjamesjacoby
11 years ago

In 5220:

In bbp_kses_allowed_tags() use true instead of empty array() for individual tag attributes. Props netweb. See #2483.

Note: See TracTickets for help on using tickets.