Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#27417 closed defect (bug) (fixed)

recent posts widget default number of posts to show

Reported by: afercia's profile afercia Owned by: ocean90's profile ocean90
Milestone: 3.9 Priority: low
Severity: normal Version: 2.8
Component: Widgets Keywords: has-patch
Focuses: ui Cc:

Description

noticeable especially in the new widgets customizer preview:
in the field "Number of posts to show" the default value is 5
but in the preview you will see 10 posts by default
proposed patch attached

Attachments (2)

default-widgets.php.patch (905 bytes) - added by afercia 11 years ago.
27417.2.patch (4.6 KB) - added by westonruter 11 years ago.
Use DEFAULT_NUMBER class constant; prevent customizer default input styles from applying to widget control forms; use input number type for integer widget fields and ensure proper number styles. Commits also pushed to GitHub branch: https://github.com/x-team/wordpress-develop/compare/trac-27417

Download all attachments as: .zip

Change History (4)

@westonruter
11 years ago

Use DEFAULT_NUMBER class constant; prevent customizer default input styles from applying to widget control forms; use input number type for integer widget fields and ensure proper number styles. Commits also pushed to GitHub branch: https://github.com/x-team/wordpress-develop/compare/trac-27417

#1 @westonruter
11 years ago

  • Focuses ui added
  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 3.9
  • Priority changed from normal to low
  • Version changed from trunk to 2.8

Good catch, @afercia! I confirm that when first inserting a Recent Posts widget, it shows 10 posts in the preview. Then when starting to edit the title, it then receives the default input value of 5. I discovered a couple other issues related to widgets in the customizer and amended the patch with those fixes.

#2 @ocean90
11 years ago

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

In 27561:

In Recent Posts widget set default number to 5.

props afercia.
fixes #27417.

Note: See TracTickets for help on using tickets.