Opened 7 years ago

Closed 7 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: Keywords: has-patch


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>
	<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 />

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

	<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 />

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 7 years ago.

Download all attachments as: .zip

Change History (2)

7 years ago

#1 @netweb
7 years ago

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

In 5119: Use consistent markup in form-topic.php.

Props SergeyBiryukov.
Fixes #2568.

Note: See TracTickets for help on using tickets.