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] Update paragraph text sizes across remaining OSD #7603

Merged

Conversation

danieldong51
Copy link
Contributor

@danieldong51 danieldong51 commented Jul 31, 2024

Description

This PR updates normal paragraph font size to be small (15.75px next theme / 14px V7 theme).

Screenshot

Data Sources

Area Before (Inspect) After (Inspect) After
Access Description Access Control v7Light Before Inspect Access Control v7Light Post Inspect Access Control v7Light Post
Accelerations Description Accelerations v7Light Before Inspect Accelerations v7Light Post Inspect Accelerations v7Light Post
Associated Objects Description Associated Objects Before Inspect Associated Objects Post Inspect Associated Objects Post
Authentication Description Authentication Before Inspect Authentication Post Inspect Authentication Post
No Data Source Message Connection v7Light Before Inspect Connection v7Light Post Inspect Connection v7Light Post
Create: Select Source Description Create Before Inspect Create Post Inspect Create Post
Create: Page Description Create_Page Before Inspect Create_Page Post Inspect Create_Page Post
Delete Modal Delete Before Inspect Delete Post Inspect Delete Post
Permissions Page Description Permissions Before Inspect Permissions Post Inspect Permissions Post
Data Sources: Description Table Before Inspect Table Post Inspect Table Post

Dev Tools

Note that the Help change was applied to the entire flyout, excluding the headers (so the commands in bold below are also under <EuiText size='s'>)

Area Before (Inspect) After (Inspect) After
Help Help Before Inspect Help Post 2v Inspect Help Post 2v
History History Before Inspect History Post Inspect History Post

Index Patterns

Note that the Scripted Field: Edit change was applied to both the description and the link below.

Area Before (Inspect) After (Inspect) After
Scripted Field: Edit Scripted Field v7Light Before Inspect Scripted Field v7Light Post Inspect Scripted Field v7Light Post
Details Details Before Inspect Details Post Inspect Details Post
Scripted Fields Scripted Fields Before Inspect Scripted Fields Post Inspect Scripted Fields Post
Source Filters Source Filters Before Inspect Source Filters Post Inspect Source Filters Post
Index Pattern Table Table Before Inspect Table Post Inspect Table Post

Other Pages

Area Before (Inspect) After (Inspect) After
Dashboard Listing Dashboards Listing Before Inspect Dashboards Listing Post Inspect Dashboards Listing Post

Changelog

  • Refactor: [Look&Feel] Update paragraph text sizes across remaining OSD

Check List

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

Copy link
Contributor

❌ Empty Changelog Section

The Changelog section in your PR description is empty. Please add a valid changelog entry or entries. If you did add a changelog entry, check to make sure that it was not accidentally included inside the comment block in the Changelog section.

Copy link
Contributor

❌ Empty Changelog Section

The Changelog section in your PR description is empty. Please add a valid changelog entry or entries. If you did add a changelog entry, check to make sure that it was not accidentally included inside the comment block in the Changelog section.

2 similar comments
Copy link
Contributor

❌ Empty Changelog Section

The Changelog section in your PR description is empty. Please add a valid changelog entry or entries. If you did add a changelog entry, check to make sure that it was not accidentally included inside the comment block in the Changelog section.

Copy link
Contributor

❌ Empty Changelog Section

The Changelog section in your PR description is empty. Please add a valid changelog entry or entries. If you did add a changelog entry, check to make sure that it was not accidentally included inside the comment block in the Changelog section.

Copy link

codecov bot commented Jul 31, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.66%. Comparing base (d09895a) to head (75e1c69).
Report is 228 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7603      +/-   ##
==========================================
+ Coverage   63.64%   63.66%   +0.01%     
==========================================
  Files        3632     3632              
  Lines       80043    80060      +17     
  Branches    12677    12683       +6     
==========================================
+ Hits        50943    50969      +26     
+ Misses      25993    25986       -7     
+ Partials     3107     3105       -2     
Flag Coverage Δ
Linux_1 30.55% <0.00%> (+0.01%) ⬆️
Linux_2 55.58% <ø> (ø)
Linux_3 40.56% <100.00%> (?)
Linux_4 31.30% <0.00%> (+<0.01%) ⬆️
Windows_1 30.57% <0.00%> (+0.01%) ⬆️
Windows_2 55.53% <ø> (ø)
Windows_3 40.56% <100.00%> (+<0.01%) ⬆️
Windows_4 31.30% <0.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

❌ Empty Entry Description

Description for "Skip" entry cannot be empty.

3 similar comments
Copy link
Contributor

❌ Empty Entry Description

Description for "Skip" entry cannot be empty.

Copy link
Contributor

github-actions bot commented Aug 1, 2024

❌ Empty Entry Description

Description for "Skip" entry cannot be empty.

Copy link
Contributor

github-actions bot commented Aug 1, 2024

❌ Empty Entry Description

Description for "Skip" entry cannot be empty.

opensearch-changeset-bot bot added a commit to danieldong51/OpenSearch-Dashboards that referenced this pull request Aug 1, 2024
danieldong51 pushed a commit to danieldong51/OpenSearch-Dashboards that referenced this pull request Aug 1, 2024
@danieldong51 danieldong51 marked this pull request as ready for review August 1, 2024 22:15
Copy link
Collaborator

@virajsanghvi virajsanghvi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add some tests (likely just snapshot tests) to meet patch codecov check?

@danieldong51 danieldong51 changed the title [Look&Feel] Update paragraph text sizes across remaining OSD [Look&Feel] Update paragraph text sizes across remaining experiences Aug 2, 2024
opensearch-changeset-bot bot added a commit to danieldong51/OpenSearch-Dashboards that referenced this pull request Aug 2, 2024
opensearch-changeset-bot bot added a commit to danieldong51/OpenSearch-Dashboards that referenced this pull request Aug 2, 2024
@danieldong51 danieldong51 changed the title [Look&Feel] Update paragraph text sizes across remaining experiences [Look&Feel] Update paragraph text sizes across remaining OSD Aug 2, 2024
@zhongnansu zhongnansu added backport 2.x v2.17.0 look & feel Look and Feel Improvements labels Aug 5, 2024
@zhongnansu zhongnansu merged commit 43dc2d6 into opensearch-project:main Aug 5, 2024
73 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Aug 5, 2024
* Updated Data Source Connection paragraph size

Signed-off-by: Dan Dong <[email protected]>

* Updated paragraph size

Signed-off-by: Dan Dong <[email protected]>

* Updated paragraph size

Signed-off-by: Dan Dong <[email protected]>

* Updated paragraph size

Signed-off-by: Dan Dong <[email protected]>

* Changeset file for PR #7603 created/updated

* Removed paragraph tags

Signed-off-by: Dan Dong <[email protected]>

* Updated Snapshot Testing

Signed-off-by: Dan Dong <[email protected]>

* Updated snapshot testing

Signed-off-by: Dan Dong <[email protected]>

* Updated help panel

Signed-off-by: Dan Dong <[email protected]>

* Updated Snapshot Testing for get_no_items_message

Signed-off-by: Dan Dong <[email protected]>

* Updated Snapshot Testing for get_no_items_message

Signed-off-by: Dan Dong <[email protected]>

* Changeset file for PR #7603 created/updated

* Changeset file for PR #7603 created/updated

* Updated Testing for get_no_items_message

Signed-off-by: Dan Dong <[email protected]>

---------

Signed-off-by: Dan Dong <[email protected]>
Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
(cherry picked from commit 43dc2d6)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
virajsanghvi pushed a commit that referenced this pull request Aug 7, 2024
…7622)

---------
(cherry picked from commit 43dc2d6)

Signed-off-by: Dan Dong <[email protected]>
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: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[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.

4 participants