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

[META] Dashboards & Plugins OUI Compliance #4065

Open
1 of 8 tasks
seanneumann opened this issue May 18, 2023 · 0 comments
Open
1 of 8 tasks

[META] Dashboards & Plugins OUI Compliance #4065

seanneumann opened this issue May 18, 2023 · 0 comments
Assignees
Labels
enhancement New feature or request OUI compliance Issues and PRs to maximize OUI usage and remove style and component hacks OUI Issues that require migration to OUI

Comments

@seanneumann
Copy link
Contributor

seanneumann commented May 18, 2023

We want to eventually do a complete end-to-end redesign of OpenSearch Dashboards. In order for this redesign to be scalable and consistent, it will leverage themes provided by OUI.

Today, OpenSearch Dashboards and OpenSearch Dashboard plugins don't always use OUI, or use it properly (as documented in the usage guidelines). There are many UI implementations that use overrides or custom styles / components. In order for theming updates to be applied correctly and consistently, a prerequisite is to remove any potentially conflicting styles and components.

Our goal is to make styling and component definition the sole responsibility of OUI. An idealized outcome would be to remove all inline styles, classes, and stylesheets from OpenSearch Dashboards.

Achieving and maintaining OUI compliance will have several work streams:

The general lifecycle of audits (per plugin or core service):

flowchart TD
a["Audit"] --> r1["Recommendation 1"]
    a --> r2["Recommendation 2"]
    a --> rellip["..."]
    a --> rn["Recommendation N"]
    r1 --> m1["Mitigation 1"]
    r2 --> m2["Mitigation 2"]
    rn --> mn["Mitigation N"]
    r1 --> rr1["Review and align"]
    rr1 --> r1
    r2 --> rr2["Review and align"]
    rr2 --> r2
    rn --> rrn["Review and align"]
    rrn --> rn
    rellip --> mellip["..."]
    m1 --> ux["UX Review and Validation"]
    m2 --> ux
    mellip --> ux
    mn --> ux
    ux --> cc["Compliance Complete"]
Loading

In addition to this META issue, we will be tracking OUI compliance changes in this GitHub Project. We are also labeling issues with OUI compliance Issues and PRs to maximize OUI usage and remove style and component hacks .

@seanneumann seanneumann added the enhancement New feature or request label May 18, 2023
@seanneumann seanneumann self-assigned this May 18, 2023
@seanneumann seanneumann pinned this issue May 18, 2023
@rednaksi91 rednaksi91 moved this to Todo in Look & Feel May 18, 2023
@joshuarrrr joshuarrrr added the OUI Issues that require migration to OUI label May 23, 2023
@joshuarrrr joshuarrrr added the OUI compliance Issues and PRs to maximize OUI usage and remove style and component hacks label Jun 1, 2023
@joshuarrrr joshuarrrr moved this from Todo to In Progress in Look & Feel Jun 1, 2023
@seanneumann seanneumann unpinned this issue Jun 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request OUI compliance Issues and PRs to maximize OUI usage and remove style and component hacks OUI Issues that require migration to OUI
Projects
Status: In Progress
Development

No branches or pull requests

5 participants