Making WordPress.org

Opened 8 years ago

Closed 8 years ago

#2568 closed defect (bug) (fixed)

bb-base: Update form-topic.php for consistency with form-reply.php

Reported by: sergeybiryukov's profile SergeyBiryukov Owned by: netweb's profile netweb
Milestone: Priority: normal
Component: bbpress.org Keywords: has-patch
Cc:

Description

Background: #2566

In bb-base and related themes, there's a markup inconsistency between form-topic.php and form-reply.php.

In form-topic.php, the <legend> element contains only the Revision label:

<legend><?php _e( 'Revision', 'wporg-forums' ); ?></legend>
<div>
	<input name="bbp_log_topic_edit" id="bbp_log_topic_edit" type="checkbox" value="1" <?php bbp_form_topic_log_edit(); ?> />
	<label for="bbp_log_topic_edit"><?php _e( 'Keep a log of this edit:', 'wporg-forums' ); ?></label><br />
</div>

In form-reply.php, the <legend> element contains the whole input:

<legend>
	<input name="bbp_log_reply_edit" id="bbp_log_reply_edit" type="checkbox" value="1" <?php bbp_form_reply_log_edit(); ?> />
	<label for="bbp_log_reply_edit"><?php esc_html_e( 'Keep a log of this edit:', 'wporg-forums' ); ?></label><br />
</legend>

Looks like form-topic.php should be updated to use the same markup, see [bb5060].

Attachments (1)

2568.patch (3.1 KB) - added by SergeyBiryukov 8 years ago.

Download all attachments as: .zip

Change History (2)

@SergeyBiryukov
8 years ago

#1 @netweb
8 years ago

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

In 5119:

bbPress.org/BuddyPress.org: Use consistent markup in form-topic.php.

Props SergeyBiryukov.
Fixes #2568.

Note: See TracTickets for help on using tickets.