-
Notifications
You must be signed in to change notification settings - Fork 871
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
[Console] Replace usage of PanelsContainer
with OuiResizableContainer
.
#4273
Comments
I wish to work on this issue. |
After looking into it, I came to conclusion that this issue is blocked until we move from Another important finding is that
In other cases default_editor is used, which uses PanelsContainer under the hood.OpenSearch-Dashboards/src/plugins/vis_default_editor/public/default_editor.tsx Lines 108 to 136 in b337bea
Example of plugins affected: vis_type_markdown , vis_type_markdown . It is likely any plugin except vis_builder uses it instead of OuiResizableContainer
I did not find other |
After conducting the audit of the Console plugin #3966, one of the action item was to remove
PanelsContainer
usage in favor ofOuiResizableContainer
for better OUI compliance.OpenSearch-Dashboards/src/plugins/console/public/application/containers/editor/editor.tsx
Lines 92 to 109 in ca0bb8f
This issue tracks the progress on this recommendation.
The text was updated successfully, but these errors were encountered: