Skip to content

[WIP] LG-5026: Drawer Toolbar Integration 🛠️ #2866

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

shaneeza
Copy link
Collaborator

@shaneeza shaneeza commented May 22, 2025

No description provided.

* add toolbar

* Updates Readme (#2806)

* rm create-leafygreen-app

* Update README.md

* lint

* add keyboard focus management

* fix focus hijacking

* naming

* updates types, fixes focus on click

* Toolbar focus state

* fix error

* add stories

* toolbar generated stories

* ToolbarIconButton generated stories

* update comments

* add testids and add an error if trying to use ToolbarIconButton outside of Toolbar

* interaction tests

* WIP test harnesses

* test harnesses

* toolbar specs

* ToolbarIconButton specs

* lint

* pass className to styles

* README

* add missing deps

* use test utils in interaction tests

* changesets

* remove darkMode from context

* Use latest version of `useBackdropClick` (#2810)

* upate hooks

* with changeset

* revert allowProp

* fix ts error

* some cleanup

* add ref test

* use correct test

* remove iconButton changes

* hellllllo

* fix build error

* Storybook export `findStories` (#2808)

* update webpack

* findStories args

* changeset

* Update index.ts

* revert to getLGConfig

* Updates `lg install` [LG-5040, LG-5041] (#2807)

* updates lg install script

* updates meta

* adds --dry option to install command

* changesets

* adds package set flags

* cli minor

* fix install pkg prebuild

* adds tests

* updates lint scrips

* lint all_packages file on write

* run lint

* Update getLGConfig.ts

* Update getPackagesToInstall.ts

* Update getPackagesToInstall.spec.ts

* fix mocks

* update flags to be additive

* push not concat

* Update getLGConfig.spec.ts

* Update package.json

* adds separate files for ESSENTIALS & BASIC

* --ui

* readonly<>

* fix tests

* Update pnpm-lock.yaml

* add missing lodash

* adds two new icons (#2815)

* constants

* update README

* Version Packages (#2811)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* build error and export const

* move handle click to context

* unique test ids for toolbarIconButton

* move test harnesses to testing dir

* aria label check

* update README

* Add more descriptive labels to some toolbar icons

---------

Co-authored-by: Adam Thompson <[email protected]>
Co-authored-by: Brooke Scarlett Yalof <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Copy link

changeset-bot bot commented May 22, 2025

🦋 Changeset detected

Latest commit: e2043f4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 34 packages
Name Type
@leafygreen-ui/toolbar Patch
@leafygreen-ui/icon-button Patch
@leafygreen-ui/drawer Patch
@lg-charts/chart-card Patch
@lg-charts/core Patch
@lg-chat/message-feedback Patch
@lg-chat/title-bar Patch
@leafygreen-ui/banner Patch
@leafygreen-ui/code Patch
@leafygreen-ui/combobox Patch
@leafygreen-ui/date-picker Patch
@leafygreen-ui/expandable-card Patch
@leafygreen-ui/guide-cue Patch
@leafygreen-ui/menu Patch
@leafygreen-ui/modal Patch
@leafygreen-ui/pagination Patch
@leafygreen-ui/password-input Patch
@leafygreen-ui/search-input Patch
@leafygreen-ui/table Patch
@leafygreen-ui/toast Patch
@lg-charts/drag-provider Patch
@lg-chat/chat-window Patch
@lg-chat/fixed-chat-window Patch
@lg-chat/message Patch
@leafygreen-ui/form-footer Patch
@lg-tools/storybook-decorators Patch
@lg-chat/lg-markdown Patch
@leafygreen-ui/split-button Patch
@lg-chat/chat-disclaimer Patch
@leafygreen-ui/confirmation-modal Patch
@leafygreen-ui/marketing-modal Patch
@lg-chat/input-bar Patch
@lg-chat/message-feed Patch
@lg-tools/storybook-addon Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@TheSonOfThomp TheSonOfThomp added the Integration 🔗 labels an integration branch label Jun 6, 2025
shaneeza added 3 commits June 6, 2025 15:35
* add toolbar

* add keyboard focus management

* fix focus hijacking

* naming

* updates types, fixes focus on click

* Toolbar focus state

* fix error

* add stories

* toolbar generated stories

* ToolbarIconButton generated stories

* update comments

* add testids and add an error if trying to use ToolbarIconButton outside of Toolbar

* interaction tests

* WIP test harnesses

* test harnesses

* toolbar specs

* ToolbarIconButton specs

* lint

* pass className to styles

* README

* add missing deps

* use test utils in interaction tests

* changesets

* remove darkMode from context

* fix ts error

* some cleanup

* add ref test

* use correct test

* remove iconButton changes

* hellllllo

* fix build error

* transition styles

* wip, transitiob broken

* constants

* update README

* wip, translate grid, still not working

* build error and export const

* move handle click to context

* unique test ids for toolbarIconButton

* move test harnesses to testing dir

* aria label check

* update README

* toolbar layout, wip

* standalone overlay not working

* add back inner drawer styles

* scrolling and shadows

* remove some comments and shouldAnimate

* clean up embeddedDrawerLayout

* comments

* overlayDrawerLayout cleanup

* constants for grid areas

* shadow transition

* rename removeShadowStyles

* remove some comments

* remove some styles

* move to DrawerToolbarLayout

* update stories

* controls to exclude

* update stories

* Add more descriptive labels to some toolbar icons

* clean up

* tsdocs for LayoutData

* fix grid columns on mobile and remove overflow from story

* update stories

* wip, position absolute standalone drawer animtion

* wip: adjust keyframe timing in drawer animation for smoother transition

* feat: enhance drawer styles with border adjustments and animation fixes

* add opacity to inner container, add shouldAnimate check

* added some clarifying comments about styles

* fix tests and run lint

* refactor: improve transition properties and ensure consistent animation behavior for drawer components

* refactor: rename type for clarity and remove unused styles file

* update README

* add temp height to cloudnav story

* remove comment and pass args in stories

* update stories so arg does not override displayMode

* add comment about story border

* remove height transition

* sandy feedback, semibold drawer title

* refactor: update height properties to use 100% for consistency across drawer layouts

* remove comment

* add back visibility hidden and focus close button

* add a comment about the shadow padding

* add exports

* refactor: reorganize imports and export constants

* lint

* remove redundant padding from drawer styles

* remove green border from story

* update constants

* fix build error

* specify embedded transitions, combine embedded and overlay types

* refactor(drawer): consolidate drawer layout components

* refactor(drawer): remove deprecated DrawerTabs component and related files

* remove DrawerContext

* remove comment in drawer stack

* fix(drawer): update default drawer index

* refactor(drawer): enhance exports and update warning message for DrawerLayout

* refactor(drawer): update README and types for clarity on display modes and remove initial open state

* refactor(drawer): reorder exports
* add toolbar

* add keyboard focus management

* fix focus hijacking

* naming

* updates types, fixes focus on click

* Toolbar focus state

* fix error

* add stories

* toolbar generated stories

* ToolbarIconButton generated stories

* update comments

* add testids and add an error if trying to use ToolbarIconButton outside of Toolbar

* interaction tests

* WIP test harnesses

* test harnesses

* toolbar specs

* ToolbarIconButton specs

* lint

* pass className to styles

* README

* add missing deps

* use test utils in interaction tests

* changesets

* remove darkMode from context

* fix ts error

* some cleanup

* add ref test

* use correct test

* remove iconButton changes

* hellllllo

* fix build error

* transition styles

* wip, transitiob broken

* constants

* update README

* wip, translate grid, still not working

* build error and export const

* move handle click to context

* unique test ids for toolbarIconButton

* move test harnesses to testing dir

* aria label check

* update README

* toolbar layout, wip

* standalone overlay not working

* add back inner drawer styles

* scrolling and shadows

* remove some comments and shouldAnimate

* clean up embeddedDrawerLayout

* comments

* overlayDrawerLayout cleanup

* constants for grid areas

* shadow transition

* rename removeShadowStyles

* remove some comments

* remove some styles

* move to DrawerToolbarLayout

* update stories

* controls to exclude

* update stories

* Add more descriptive labels to some toolbar icons

* provider with tests

* add imports

* refactor DrawerToolbarLayout to use context

* update DrawerToolbarLayout so content and title are optional

* wip, test ids for testing

* update test harnesses

* update label in story

* clean up

* tsdocs for LayoutData

* fix grid columns on mobile and remove overflow from story

* fix grid columns on mobile and remove overflow from story

* update stories

* add back plus

* update long content

* update layout stories and add interaction tests

* interaction to Interaction story name

* wip, position absolute standalone drawer animtion

* wip: adjust keyframe timing in drawer animation for smoother transition

* feat: enhance drawer styles with border adjustments and animation fixes

* Enhance drawer animations and state management

* add opacity to inner container, add shouldAnimate check

* added some clarifying comments about styles

* fix tests and run lint

* refactor: improve transition properties and ensure consistent animation behavior for drawer components

* refactor: rename type for clarity and remove unused styles file

* update drawer toolbar context tsdoc, lint

* refactor: update import paths and enhance drawer toolbar context tests

* refactor: streamline import statements and add pauses for visibility in interaction stories

* update README

* add DrawerToolbarLayout to README

* update README

* add temp height to cloudnav story

* remove comment and pass args in stories

* update stories so arg does not override displayMode

* add comment about story border

* remove height transition

* pass rest in story

* sandy feedback, semibold drawer title

* add onClose to README, add exports, fix eslint warnings

* add note on height to README

* add note on data and toolbar items in README

* some more README updates

* refactor: update height properties to use 100% for consistency across drawer layouts

* remove comment

* replace shouldCloseDrawer with isDrawerOpen, replace getActiveDrawerContent with activeDrawerContent

* forgot to save file

* refactor: rename variable getActiveDrawerContent to activeDrawerContent

* add back visibility hidden and focus close button

* add a comment about the shadow padding

* add exports

* refactor: reorganize imports and export constants

* lint

* remove redundant padding from drawer styles

* remove green border from story

* update constants

* fix build error

* specify embedded transitions, combine embedded and overlay types

* refactor(drawer): consolidate drawer layout components

* refactor(drawer): remove deprecated DrawerTabs component and related files

* remove DrawerContext

* remove comment in drawer stack

* fix(drawer): update default drawer index

* refactor(drawer): enhance exports and update warning message for DrawerLayout

* refactor(drawer): update README and types for clarity on display modes and remove initial open state

* refactor(drawer): reorder exports

* run lint

* fix failing ui tests and merge conflicts
@mongodb mongodb deleted a comment from github-actions bot Jun 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Integration 🔗 labels an integration branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants