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

refactor: increasing the width of topology tooltip #2395

Merged
merged 2 commits into from
Sep 13, 2023

Conversation

anandtiwary
Copy link
Contributor

@anandtiwary anandtiwary commented Sep 12, 2023

Description

Please include a summary of the change, motivation and context.
refactor: increasing the width of topology tooltip

Testing

Please describe the tests that you ran to verify your changes. Please summarize what did you test and what needs to be tested e.g. deployed and tested helm chart locally.

Checklist:

  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • Any dependent changes have been merged and published in downstream modules

Documentation

Make sure that you have documented corresponding changes in this repository or hypertrace docs repo if required.

@anandtiwary anandtiwary requested a review from a team as a code owner September 12, 2023 21:55
@codecov
Copy link

codecov bot commented Sep 12, 2023

Codecov Report

Merging #2395 (e32a40f) into main (49dbc75) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2395   +/-   ##
=======================================
  Coverage   82.73%   82.73%           
=======================================
  Files         920      920           
  Lines       20510    20510           
  Branches     3234     3234           
=======================================
  Hits        16968    16968           
  Misses       3413     3413           
  Partials      129      129           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@github-actions

This comment has been minimized.

@anandtiwary anandtiwary requested review from aaron-steinfeld and a team September 13, 2023 01:09
@@ -7,7 +7,7 @@
opacity: 0.92;
box-shadow: 0px 4px 5px rgba(0, 0, 0, 0.2), 0px 3px 14px rgba(0, 0, 0, 0.12), 0px 8px 10px rgba(0, 0, 0, 0.14);
border-radius: 4px;
max-width: 400px;
max-width: 800px;
Copy link
Contributor

Choose a reason for hiding this comment

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

should we do this or wrap the text below instead of the ellipsis?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is fine :)

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@anandtiwary anandtiwary merged commit eae893f into main Sep 13, 2023
25 checks passed
@anandtiwary anandtiwary deleted the setting-larger-max-width-for-tooltip branch September 13, 2023 23:08
@github-actions
Copy link

Unit Test Results

       4 files     309 suites   45m 26s ⏱️
1 120 tests 1 120 ✔️ 0 💤 0 ❌
1 130 runs  1 130 ✔️ 0 💤 0 ❌

Results for commit eae893f.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants