I have a use case where I am using several jump menus on a single page, however the use of "Navigation" as the default value for it is too generic and can confuse a user when they see it multiple times on a page. It would be great to add the ability for a user to define the default as the block's title, alongside the "Show header instead of the active item" option. Thoughts?

CommentFileSizeAuthor
#2 tinynav-use_block_title_text-1808954-2.patch438 bytespdrake

Comments

btopro’s picture

btopro commented

Hmm.. so you're saying change the "Navigation" text that's injected arbitrarily with the block's title?

This way if the block's title was "Main Menu" or "secondary menu" that that would be what's displayed. I think this would be welcomed functionality in conjunction with #1797924: Allow for multiple selector / media query combinations. I'd welcome patches for both, really bogged down in projects atm.

pdrake’s picture

pdrake commented
Status: Active » Needs review
StatusFileSize
new tinynav-use_block_title_text-1808954-2.patch438 bytes

Here's a patch that replaces "Navigation" with the block title text.

pixelsweatshop’s picture

pixelsweatshop commented
Status: Reviewed & tested by the community » Fixed

I see this is already fixed in the latest dev (works well), so I am going to close this. Thanks for adding this feature. I don't see a commit message but I see the following in the forked js

I lied. I was looking at a patched version. Tested patch in #2 and it works like a charm.

pixelsweatshop’s picture

pixelsweatshop commented
Status: Needs review » Reviewed & tested by the community

Status: Fixed » Closed (fixed)

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