Skip to content
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

[Research] OUI Compliance audit of the opensearch_dashboards_react plugin #4089

Open
BSFishy opened this issue May 22, 2023 · 1 comment
Open
Labels
good first issue Good for newcomers OUI compliance Issues and PRs to maximize OUI usage and remove style and component hacks OUI Issues that require migration to OUI

Comments

@BSFishy
Copy link
Contributor

BSFishy commented May 22, 2023

Tracking issue for OUI compliance audit of the opensearch_dashboards_react plugin.

Follow the steps in #3945 to complete the audit, then open another issue with findings. Once the work of performing the audit is done, this issue can be closed.

Related files
size (bytes) file notes
6.4K src/plugins/opensearch_dashboards_react/public/markdown/_markdown.scss
1.4K src/plugins/opensearch_dashboards_react/public/overview_page/overview_page_header/_overview_page_header.scss
1.4K src/plugins/opensearch_dashboards_react/public/field_button/field_button.scss
1.2K src/plugins/opensearch_dashboards_react/public/overview_page/overview_page_header/index.scss
833 src/plugins/opensearch_dashboards_react/public/exit_full_screen_button/_exit_full_screen_button.scss
155 src/plugins/opensearch_dashboards_react/public/code_editor/editor.scss
37 src/plugins/opensearch_dashboards_react/public/exit_full_screen_button/index.scss
22 src/plugins/opensearch_dashboards_react/public/markdown/index.scss

If this issue isn't already assigned to someone and you'd like to take on this work, please ping @BSFishy in the comments and I can assign it to you.

@BSFishy BSFishy added OUI Issues that require migration to OUI and removed untriaged labels May 22, 2023
@BSFishy BSFishy added the good first issue Good for newcomers label May 24, 2023
@joshuarrrr joshuarrrr added the OUI compliance Issues and PRs to maximize OUI usage and remove style and component hacks label Jun 1, 2023
@curq
Copy link
Collaborator

curq commented Jun 9, 2023

From the audit #3966 we found out that PanelsContainer from


have OUI Component that replaces it OuiResizableContainer. So, we probably should remove it after we replace all instances where PanelsContainer is used.

Leaving this for additional context.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers OUI compliance Issues and PRs to maximize OUI usage and remove style and component hacks OUI Issues that require migration to OUI
Projects
Status: Todo
Development

No branches or pull requests

3 participants