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

feat: Add V9 Theme #1344

Merged
merged 3 commits into from
Aug 16, 2024
Merged

Conversation

virajsanghvi
Copy link
Collaborator

@virajsanghvi virajsanghvi commented Aug 15, 2024

Description

This introduces a new Theme, temporarily named V9. This is a work in progress, but putting out a PR so we can start testing it against OSD.

Most of this was copied from oui-next, but changes include:

  • new colors
  • new font
  • new type ramp
  • introducing a button border radius token across all themes (defaulted to existing value in existing themes)
  • turning on shadows for compressed inputs

Main risk with these theme at the moment are header font sizes as they render at a different size than v7/v8 counterparts.

Screenshot:
image
image

Issues Resolved

N/A

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • All tests pass
    • yarn lint
    • yarn test-unit
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Viraj Sanghvi <virajs@amazon.com>
Signed-off-by: Viraj Sanghvi <virajs@amazon.com>
Signed-off-by: Viraj Sanghvi <virajs@amazon.com>
@virajsanghvi
Copy link
Collaborator Author

Note: failing test looks unrelated (due to a workspace issue)

@AMoo-Miki AMoo-Miki merged commit a45811b into opensearch-project:main Aug 16, 2024
24 of 25 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 16, 2024
* feat: Add V9 Theme

Signed-off-by: Viraj Sanghvi <virajs@amazon.com>

* update changelog

Signed-off-by: Viraj Sanghvi <virajs@amazon.com>

* update colors and typography

Signed-off-by: Viraj Sanghvi <virajs@amazon.com>

---------

Signed-off-by: Viraj Sanghvi <virajs@amazon.com>
(cherry picked from commit a45811b)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
AMoo-Miki pushed a commit that referenced this pull request Aug 16, 2024
* feat: Add V9 Theme



* update changelog



* update colors and typography



---------


(cherry picked from commit a45811b)

Signed-off-by: Viraj Sanghvi <virajs@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants