When using the summary jump menu, I thought it might be more intuitive to show the user which argument is selected before they go to select a different argument value. I found a way of doing this that seems to be pretty effective and I'm going to attach this as a patch once I file this issue.

Comments

chromix’s picture

StatusFileSize
new1.28 KB

Here's the patch.

chromix’s picture

Status: Active » Needs review

changing status to 'needs review.'

merlinofchaos’s picture

This needs to be optional. It would change behavior in a way that might not work well for sites relying on the existing behavior.

chromix’s picture

StatusFileSize
new1.88 KB

Good call sir! This patch makes the feature optional.

dawehner’s picture

StatusFileSize
new1.75 KB
+      'default_value' => $default_value,

$default_value is not always set. This will throw erros on d7.

Here is a new patch

chromix’s picture

StatusFileSize
new1.92 KB

Thanks dereine. After doing some more testing I realized the default "- Choose -" option would never appear, so I fixed that error by checking a different variable. This patch seems to work pretty flawlessly, though.

dawehner’s picture

Looks better now, but i want to test it first.

dawehner’s picture

Is it just me that this cannot work logically: If there is a argument => the summary is not displayed => the jump menu is not displayed.

Is this just to much logical thinking, but i does not work here.

merlinofchaos’s picture

dereine: If you have the summary in a block or something it will have no arguments. You could also have it as an attachment, like the glossary view done as a select widget.

chromix’s picture

merlinofchaos: that's exactly how I'm using the jump menu (i.e. as an attachment). By using the jump menu in this way, it ends up functioning more like an exposed filter, which is exactly what I wanted.

dboulet’s picture

That's how I expected the dropdown menu to work as well—having "-Choose-" displayed after you've selected an option seems counter-intuitive.

merlinofchaos’s picture

Status: Needs review » Fixed

I extended this to work with both the main jump menu style, plus the jump menu summary like this one. This is a nice feature. Committed to all branches.

Status: Fixed » Closed (fixed)

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