Skip to content

Components: promote g2 Popover as Flyout #32197

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.

Sign up for GitHub

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

Merged
ciampo merged 25 commits into from
Jul 6, 2021
Merged

Components: promote g2 Popover as Flyout #32197

Changes from 1 commit
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
e2efe53
Move `components/src/ui/popover` to `components/src/accessible-popover`
ciampo May 25, 2021
650deeb
Add to Docs manifest
ciampo May 25, 2021
777f822
Add to tsconfig.json
ciampo May 25, 2021
8363483
Rename context-related variables from Popover* to AccessiblePopover*
ciampo Jun 25, 2021
8e0d236
Rename `PopoverContext` type to `Context`
ciampo Jun 25, 2021
59c21e4
Rename `PopoverContent` to `AccessiblePopoverContent`
ciampo Jun 25, 2021
935bc5e
Move `component.js` to component-specific folder
ciampo Jun 25, 2021
ed58977
Move `AccessibilePopover` logic to separate hook
ciampo Jun 25, 2021
083fe44
Move `content.js` to component-specific folder
ciampo Jun 25, 2021
5c15310
Move `AccessibilePopoverContent` logic to separate hook
ciampo Jun 25, 2021
1e68e3c
Rename `AccessiblePopover` to `Flyout`
ciampo Jun 25, 2021
5d3428c
Rename zIndex const from `Popover` to `Flyout`
ciampo Jun 30, 2021
2d92a1e
Do not export FlyoutContent component
ciampo Jul 1, 2021
294279d
Fix imports in example
ciampo Jul 1, 2021
337249a
Rename `popover` to `flyoutState`
ciampo Jul 1, 2021
cd5d33b
Refactor to `styled` approach
ciampo Jul 1, 2021
91857d9
Wrap `ReakitPopover` in `FlyoutContentView` and set `maxWidth` in inl…
ciampo Jul 1, 2021
8d31c25
auto-format
ciampo Jul 1, 2021
db1cbae
Add Props documentation to Flyout
ciampo Jul 1, 2021
00cc295
Mark `Flyout` as non-polymorphic
ciampo Jul 2, 2021
1b5f2a5
Add documentation to `FlyoutContent`s README
ciampo Jul 2, 2021
a7389ba
Update snapshots
ciampo Jul 2, 2021
23b945e
Remove FlyoutContext exports
ciampo Jul 2, 2021
edfb229
Do not export FlyoutContent`s documentation
ciampo Jul 2, 2021
2b5d016
Delete FlyoutContent`s README
ciampo Jul 2, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add to Docs manifest
  • Loading branch information
@ciampo
ciampo committed Jul 5, 2021
commit 650deebbae64eee2499476620ee60ccedcec817d
6 changes: 6 additions & 0 deletions docs/manifest.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -653,6 +653,12 @@
"markdown_source": "../packages/components/README.md",
"parent": "reference-guides"
},
{
"title": "AccessiblePopover",
"slug": "accessible-popover",
"markdown_source": "../packages/components/src/accessible-popover/README.md",
"parent": "components"
},
{
"title": "AlignmentMatrixControl",
"slug": "alignment-matrix-control",
Expand Down