-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[Presentation] Deprecate Solution Toolbar Button in favor of Shared UX Component #126916
Changes from 10 commits
6164988
a1b8b49
ce333b0
5c6cd01
cd0b90f
ed176d8
997b379
a75f4b9
c1755c7
5e4ea0c
8959416
7054dee
a78e943
a6975c5
d565815
6983822
18688d1
0d32c06
52081d2
46c79b0
93b5a22
e414da2
17d8b8e
d4e7ea9
ccdfdb5
8678625
49b35eb
b60cb4d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
Original file line number | Diff line number | Diff line change | ||
---|---|---|---|---|
|
@@ -25,6 +25,12 @@ export const LazySolutionToolbarButton = React.lazy(() => | |||
})) | ||||
); | ||||
|
||||
export const LazySolutionToolbarButtonProps = React.lazy(() => | ||||
import('./toolbar/index').then(({ SolutionToolbarButtonProps }: any) => ({ | ||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Two things: first, this should exist.
Second, as @majagrubic pointed out: no need to tack |
||||
default: SolutionToolbarButtonProps, | ||||
})) | ||||
); | ||||
|
||||
/** | ||||
* A `ExitFullScreenButton` component that is wrapped by the `withSuspense` HOC. This component can | ||||
* be used directly by consumers and will load the `LazyExitFullScreenButton` component lazily with | ||||
|
@@ -39,6 +45,8 @@ export const ExitFullScreenButton = withSuspense(LazyExitFullScreenButton); | |||
*/ | ||||
export const SolutionToolbarButton = withSuspense(LazySolutionToolbarButton); | ||||
|
||||
export const SolutionToolbarButtonProps = withSuspense(LazySolutionToolbarButtonProps); | ||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||
|
||||
/** | ||||
* The Lazily-loaded `NoDataViews` component. Consumers should use `React.Suspennse` or the | ||||
* `withSuspense` HOC to load this component. | ||||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The plugin's name is
sharedUX
; I think that's why the build went crazy.