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] Apply small popover padding and add Oui tooltip #7523

Merged
merged 11 commits into from
Jul 30, 2024

Conversation

danieldong51
Copy link
Contributor

@danieldong51 danieldong51 commented Jul 26, 2024

Description

Applies small padding on popovers for index management page. Also adds OUI tooltip to icons to Index Pattern Source Filters table to match other tables under index management.

Screenshot

Small padding on popovers

Added the smallPopoverPadding = s attribute to the following popovers:

Scope Before (v7Light) After (v7Light) Before (v8Dark) After (v8Dark)
Saved Objects Export Saved Objects Export v7 Light Before Saved Objects Export v7 Light After Saved Objects Export v8 Dark Before Saved Objects Export v8 Dark After
Vis Augmenter Vis Augmenter v7 Light Before Vis Augmenter v7 Light Post Vis Augmenter v8 Dark Before Vis Augmenter v8 Dark Post
Recent Items Recents v7 Light Before Recents v7 Light Post Recents v8 Dark Before Recents v8 Dark Post

Tooltip Additions

Added EuiTooltips to edit and delete icons in the index patterns source filters listing to match other tables:

Scope Before (v7Light) After (v7Light) Before (v8Dark) After (v8Dark)
Index Patterns: Source Filters Edit v7 Light Before Edit v7 Light Post Edit v8 Dark Before Edit v8 Dark Post
Index Patterns: Source Filters Delete v7 Light Before Delete v7 Light Post Delete v8 Dark Before Delete v8 Dark Post

Other tables for reference, which have tooltips for edit and delete icons:

Scope Screenshot
Dashboards Listing Dashboards Listing
Visualizations Listing Visualizations Listing
Index Patterns: Scripted Fields Screenshot 2024-07-29 at 11 20 08 AM
Index Patterns: Scripted Fields Screenshot 2024-07-29 at 11 20 14 AM

Changelog

  • refactor: [Look&Feel] Apply small popover padding and add Oui tooltips

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

❌ Invalid Prefix

Invalid description prefix. Found "[Look&Feel] Use small popover padding across the board". Expected "breaking", "deprecate", "feat", "fix", "infra", "doc", "chore", "refactor", "security", "skip", or "test".

Copy link

codecov bot commented Jul 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.63%. Comparing base (8bafd8d) to head (4ec5f8b).
Report is 235 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7523   +/-   ##
=======================================
  Coverage   63.63%   63.63%           
=======================================
  Files        3630     3630           
  Lines       79549    79549           
  Branches    12613    12613           
=======================================
  Hits        50621    50621           
  Misses      25857    25857           
  Partials     3071     3071           
Flag Coverage Δ
Linux_1 30.75% <ø> (ø)
Linux_2 55.58% <ø> (ø)
Linux_3 40.21% <ø> (+<0.01%) ⬆️
Linux_4 31.55% <100.00%> (ø)
Windows_1 30.76% <ø> (ø)
Windows_2 55.53% <ø> (ø)
Windows_3 40.21% <ø> (ø)
Windows_4 31.55% <100.00%> (ø)

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.

@danieldong51 danieldong51 changed the title [Look&Feel] Use small popover padding across the board [Look&Feel] Apply small popover padding and Oui tooltip to index management pages Jul 29, 2024
Copy link
Contributor

❌ Invalid Prefix

Invalid description prefix. Found "[Look&Feel] Use small popover padding across the board". Expected "breaking", "deprecate", "feat", "fix", "infra", "doc", "chore", "refactor", "security", "skip", or "test".

opensearch-changeset-bot bot added a commit to danieldong51/OpenSearch-Dashboards that referenced this pull request Jul 29, 2024
@danieldong51 danieldong51 changed the title [Look&Feel] Apply small popover padding and Oui tooltip to index management pages [Look&Feel] Apply small popover padding and add Oui tooltip Jul 29, 2024
opensearch-changeset-bot bot added a commit to danieldong51/OpenSearch-Dashboards that referenced this pull request Jul 29, 2024
opensearch-changeset-bot bot added a commit to danieldong51/OpenSearch-Dashboards that referenced this pull request Jul 29, 2024
danieldong51 pushed a commit to danieldong51/OpenSearch-Dashboards that referenced this pull request Jul 29, 2024
danieldong51 pushed a commit to danieldong51/OpenSearch-Dashboards that referenced this pull request Jul 29, 2024
danieldong51 pushed a commit to danieldong51/OpenSearch-Dashboards that referenced this pull request Jul 29, 2024
@danieldong51 danieldong51 marked this pull request as ready for review July 29, 2024 22:53
@zhongnansu zhongnansu added look & feel Look and Feel Improvements backport 2.x labels Jul 30, 2024
@virajsanghvi virajsanghvi merged commit 33ef32d into opensearch-project:main Jul 30, 2024
72 of 73 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jul 30, 2024
* Updated Table padding size
* Added tooltips to index patterns -> source filters
* Updated recent items popover padding
* Updated vis augmenter popover padding

---------

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 33ef32d)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
LDrago27 pushed a commit that referenced this pull request Jul 30, 2024
…7580)

* Updated Table padding size
* Added tooltips to index patterns -> source filters
* Updated recent items popover padding
* Updated vis augmenter popover padding

---------



(cherry picked from commit 33ef32d)

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>
Qxisylolo pushed a commit to Qxisylolo/OpenSearch-Dashboards that referenced this pull request Aug 1, 2024
…ch-project#7523)

* Updated Table padding size
* Added tooltips to index patterns -> source filters
* Updated recent items popover padding
* Updated vis augmenter popover padding

---------

Signed-off-by: Dan Dong <[email protected]>
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