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

[Look & Feel] Consistency and Density Changes for Snapshot Management #1148

Merged
merged 3 commits into from
Sep 2, 2024

Conversation

SheyGao
Copy link
Contributor

@SheyGao SheyGao commented Aug 25, 2024

Description

Note that this PR only contains partial changes, and the other changes are made in #1157

This PR makes consistency and density improvements, including:

  1. Using small font size as normal paragraph font size through
  2. Using semantic headers (H1s on main pages, H2s on modals/flyouts) wrapped under
  3. Using Plus icons for create use cases and PlusInCricle icon for add use cases.

Screenshots

Standard Paragraph Text Size

Before After
Screenshot 2024-08-28 at 8 13 50 PM Screenshot 2024-08-28 at 8 14 29 PM
Screenshot 2024-08-28 at 8 14 53 PM Screenshot 2024-08-28 at 8 15 16 PM
Screenshot 2024-08-28 at 8 15 46 PM Screenshot 2024-08-28 at 8 16 04 PM

Semantic Headers

Before After
Screenshot 2024-08-28 at 8 18 03 PM Screenshot 2024-08-28 at 8 18 14 PM
Screenshot 2024-08-28 at 8 18 44 PM Screenshot 2024-08-28 at 8 18 56 PM
Screenshot 2024-08-28 at 8 19 24 PM Screenshot 2024-08-28 at 8 19 39 PM
Screenshot 2024-08-28 at 8 20 03 PM Screenshot 2024-08-28 at 8 20 14 PM

Plus Icon and PlusInCircle Icon

Before After
Screenshot 2024-08-28 at 8 21 03 PM Screenshot 2024-08-28 at 8 21 26 PM
Screenshot 2024-08-28 at 8 21 46 PM Screenshot 2024-08-28 at 8 22 00 PM
Screenshot 2024-08-28 at 8 22 33 PM Screenshot 2024-08-28 at 8 22 51 PM
Screenshot 2024-08-28 at 8 23 05 PM Screenshot 2024-08-28 at 8 23 20 PM
Screenshot 2024-08-28 at 8 23 41 PM Screenshot 2024-08-28 at 8 23 56 PM
Screenshot 2024-08-28 at 8 24 12 PM Screenshot 2024-08-28 at 8 24 30 PM

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: Shey Gao <sheygao@amazon.com>
@SheyGao SheyGao changed the title [Look & Feel] Consistency and Density Changes [Look & Feel] Consistency and Density Changes for Snapshot Management Aug 29, 2024
@SheyGao SheyGao marked this pull request as ready for review August 29, 2024 03:30
@RamakrishnaChilaka
Copy link
Collaborator

LGTM, retriggered cypress E2E tests.

@SuZhou-Joe
Copy link
Member

@SheyGao Could you please fix the unit test failure?

Signed-off-by: Shey Gao <sheygao@amazon.com>
@SuZhou-Joe SuZhou-Joe merged commit a7d317a into opensearch-project:main Sep 2, 2024
10 of 11 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 2, 2024
…#1148)

* initial changes

Signed-off-by: Shey Gao <sheygao@amazon.com>

* fixed unit tests

Signed-off-by: Shey Gao <sheygao@amazon.com>

---------

Signed-off-by: Shey Gao <sheygao@amazon.com>
Co-authored-by: Shey Gao <sheygao@amazon.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
(cherry picked from commit a7d317a)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
SuZhou-Joe pushed a commit that referenced this pull request Sep 2, 2024
…#1148) (#1159)

* initial changes



* fixed unit tests



---------




(cherry picked from commit a7d317a)

Signed-off-by: Shey Gao <sheygao@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>
Co-authored-by: Shey Gao <sheygao@amazon.com>
Co-authored-by: Hailong Cui <ihailong@amazon.com>
astute-decipher pushed a commit to astute-decipher/index-management-dashboards-plugin that referenced this pull request Sep 2, 2024
…opensearch-project#1148)

* initial changes

Signed-off-by: Shey Gao <sheygao@amazon.com>

* fixed unit tests

Signed-off-by: Shey Gao <sheygao@amazon.com>

---------

Signed-off-by: Shey Gao <sheygao@amazon.com>
Co-authored-by: Shey Gao <sheygao@amazon.com>
Co-authored-by: Hailong Cui <ihailong@amazon.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.

4 participants