When configuring the "list" and "unformatted" argument summaries, there is a checkbox to "Override number of items to display". However, checking this box doesn't expose the text field where site builders can enter a number:

https://skitch.com/becw/fp48b/argument-summary-pager-before

When the "Override ..." box is checked, the form should look like this:

https://skitch.com/becw/fp4gi/argument-summary-pager-after

It looks like the #dependency form property wasn't updated to the Drupal 7 #states property. Here's a patch.

Comments

merlinofchaos’s picture

Status: Needs review » Needs work

We're not actually using states; it couldn't handle every case, so we needed to retain dependency. For consistency we kept everything with #dependency.

merlinofchaos’s picture

Hmm. It looks like this is happening because the IDs changed. Unfortunately, the ID now actually embeds the summary style so it's not quite as simple to fix as it should be.

dawehner’s picture

Status: Needs work » Needs review
StatusFileSize
new1.29 KB

So here is a patch, which works fine and looks clean.

becw’s picture

Status: Needs review » Reviewed & tested by the community

Works for me, looks good. Thanks!

dawehner’s picture

Status: Reviewed & tested by the community » Fixed

Commited to 7.x-3.x

Status: Fixed » Closed (fixed)

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