Add an option to display the top level page in BookNavigationBlock

Will need test coverage. Also will need a 3.0.x branch

Test added and MR added for 3.0.x branch.

Looking pretty good, will need an upgrade hook for the new schema key.

smustgrave → closed merge request !79

This fix doesn't allow Use top-level page title as block title and Show top level item together. The new functionality takes precedence.

lets address that.

And probably expand testBookNavigationBlockWithTopLevelPageTitle to test both.

siramsay → changed the visibility of the branch 3497281-add-an-option to active.

Sorry, I don't know what I did above changed the visibility of the branch 3497281-add-an-option to active..
I was trying to find the correct issue branch and commit, so must have clicked on something. Please change back if that makes sense.

This fixes the issue so that Use top-level page title as block title and Show top level item can be use together.

Still needs test.

Tested on drupal 11 install. Placed a book navigation block. With both show_top_item and use_top_level_title enabled, both configurations work as described.

Pushed up a commit to shorten the description for show_top_item. Moving to RTBC.

May need a reroll

@siramsay can you test to see if this addresses your issue.

Okay, I can do that.

I’m hoping to have a alpha release for 3.0.x but it’s such a drastic shift I’ll backport this to 2.0.x too

This would be good to have in the stable 2.x release, since 3.0.x is the dev release. I'll test this out later today. Sorry, I should have said I couldn't get to it till today. Thanks.

0 rush

I tested the latest commit, and this works as advertised. https://git.drupalcode.org/issue/book-3497281/-/commits/c0da465fce1c465cd7f24d0c5f409b5084651f45 with composer.json set up to use this.

composer require 'drupal/book:3.0.x-dev#c0da465fce1c465cd7f24d0c5f409b5084651f45'

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