Change to when tabs are shown

[willguv]

Following on from Andy's comment

For the details group / tabs, we changed the switch between then to 1138px, so narrow viewports use detail dropdowns between the sections instead of vertical tabs.

Let's consider the impact of this change

[finnlewis]

Note; this is configuration in the tab group. Setting up vertical tabs, allows you to set the breakpoint.

[finnlewis]

Note: this would probably just be for new installs.

Tasks:

• identify all vertical tab groups in the LocalGov Drupal distribution.
• Update configuration for each to 1138px breakpoint
• possibly update the README.md for each repo
• create pull requests in each repo with recommended release notes.

[agile-simon]

Pull requests

Directories
localgovdrupal/localgov_directories#329

News
localgovdrupal/localgov_news#100

Services
localgovdrupal/localgov_services#235

Subsites
localgovdrupal/localgov_subsites#136

[willguv]

@stephen-cox please review Simon's PRs - thanks

[willguv]

@stephen-cox have you reviewed/ finished these PRs? Thanks

[stephen-cox]

Hi @willguv - these are all merged apart from localgovdrupal/localgov_services#235, which has test failures that we haven't got to the bottom of.

[alexjohntweedie]

Yes

…

On Tue, Feb 13, 2024, 2:34 PM Stephen Cox ***@***.***> wrote: Hi @willguv <https://github.com/willguv> - these are all merged apart from localgovdrupal/localgov_services#235 <localgovdrupal/localgov_services#235>, which has test failures that we haven't got to the bottom of. — Reply to this email directly, view it on GitHub <#538 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AMTROUWBEU3Q5NMXU3CMF23YTN2ZFAVCNFSM6AAAAAAXLMNAY6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBRGY2TAMJRHE> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[markconroy]

I've merged the latest 2.x into the PR at #235

All tests are passing now, and the PR is working. I'm going to mark it as approved.

===
Thanks to Big Blue Door for sponsoring my time to work on this.

[mccrodp]

Hey @markconroy, what remains as an open action in this issue following your merge above? It wasn't clear to me if anything else is required or if this can be closed now. Thanks.

[markconroy]

I think we can close this one if all the PRs are merged.