Skip to content

[4.x] Fix Bard Toolbar position when in fullscreen mode - #8024

Merged
jackmcdade merged 1 commit into
statamic:masterfrom
o1y:fix-bard-fullscreen-toolbar
Apr 29, 2023
Merged

[4.x] Fix Bard Toolbar position when in fullscreen mode#8024
jackmcdade merged 1 commit into
statamic:masterfrom
o1y:fix-bard-fullscreen-toolbar

Conversation

@o1y

@o1y o1y commented Apr 29, 2023

Copy link
Copy Markdown
Contributor

Fixes
2023-04-29 12 52 28

Bildschirmfoto 2023-04-29 um 13 04 11

@what-the-diff

what-the-diff Bot commented Apr 29, 2023

Copy link
Copy Markdown

PR Summary

  • Updated selector for the sticky toolbar
    This change ensures that only top-level Bard fields have a sticky header, avoiding issues with nested fields on pages like settings and content types. This improves the user experience by preventing unwanted fixed toolbars on nested Bard fields that do not scroll independently.

@o1y o1y changed the title Fix Bard Toolbar position when in fullscreen mode [4.x] Fix Bard Toolbar position when in fullscreen mode Apr 29, 2023
@o1y
o1y force-pushed the fix-bard-fullscreen-toolbar branch from 30300cb to f3d221b Compare April 29, 2023 11:06
@jackmcdade
jackmcdade merged commit aecfb28 into statamic:master Apr 29, 2023
@jackmcdade

Copy link
Copy Markdown
Member

Ahh dang I had the feeling i was forgetting a test case. Thank you!

@o1y
o1y deleted the fix-bard-fullscreen-toolbar branch June 20, 2026 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants