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

Remove KUI usage in opensearch_dashboards_legacy #3830

Open
Tracked by #1060
joshuarrrr opened this issue Apr 11, 2023 · 0 comments
Open
Tracked by #1060

Remove KUI usage in opensearch_dashboards_legacy #3830

joshuarrrr opened this issue Apr 11, 2023 · 0 comments
Labels
de-angular de-angularize work OUI Issues that require migration to OUI technical debt If not paid, jeapardizes long-term success and maintainability of the repository. ux / ui Improvements or additions to user experience, flows, components, UI elements

Comments

@joshuarrrr
Copy link
Member

joshuarrrr commented Apr 11, 2023

The opensearch_dashboards_legacy plugin uses a single .kui* namespaced class. See https://github.com/search?q=repo%3Aopensearch-project%2FOpenSearch-Dashboards+kui+NOT+trackUI+path%3A%2F%5Esrc%5C%2Fplugins%5C%2Fopensearch_dashboards_legacy%2F&type=code

To resolve

Investigate and inspect the applied CSS of that class and the rendered component. If necessary:

  1. Document CSS rules provided by these classes
  2. Replace these components with OUI components

Note that opensearch_dashboards_legacy still needs to be rewritten from Angular to React - #1560

It may make sense to just de-angularize first and replace with the OUI react components.

@joshuarrrr joshuarrrr added ux / ui Improvements or additions to user experience, flows, components, UI elements technical debt If not paid, jeapardizes long-term success and maintainability of the repository. de-angular de-angularize work and removed untriaged labels Apr 11, 2023
@seanneumann seanneumann added the OUI Issues that require migration to OUI label May 18, 2023
@ashwin-pc ashwin-pc removed the v2.11.0 label Oct 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
de-angular de-angularize work OUI Issues that require migration to OUI technical debt If not paid, jeapardizes long-term success and maintainability of the repository. ux / ui Improvements or additions to user experience, flows, components, UI elements
Projects
Status: Todo
Development

No branches or pull requests

3 participants