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

127 Update tooltips #137

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

127 Update tooltips #137

wants to merge 1 commit into from

Conversation

moconnell9
Copy link
Contributor

Update Tooltips

Addresses issue #127

Summary

  • Update styling of every tooltip in the app and its '?' button to match the new Figma
  • Update/add copy for 6 of the tooltips (screenshots below)

References

Updated/Added Tooltips

Outage Severity Score

Before:

(none)

After:

Screenshot 2024-05-02 at 10 36 57 PM

Total Severity Score

TODO: Need help finding this

Before:

After:

[COUNTRY NAME] Internet Connectivity

Before:

Screenshot 2024-05-02 at 10 38 43 PM

After:

Screenshot 2024-05-02 at 10 06 50 PM
Note: Regions and ASN/ISP is unchanged here:
Screenshot 2024-05-02 at 10 48 23 PM

Time Range

Before:

Screenshot 2024-05-02 at 10 38 37 PM

After:

Screenshot 2024-05-02 at 10 14 15 PM

[COUNTRY NAME] Regional Outages

Before:

Screenshot 2024-05-02 at 10 38 55 PM

After:

Screenshot 2024-05-02 at 10 10 25 PM

[COUNTRY NAME] Network Outages

Before:

Screenshot 2024-05-02 at 10 39 11 PM

After:

Screenshot 2024-05-02 at 10 12 00 PM

@moconnell9 moconnell9 requested a review from salcock May 3, 2024 03:14
@moconnell9 moconnell9 marked this pull request as draft May 3, 2024 03:14
@moconnell9 moconnell9 force-pushed the 127-update-tooltips branch 2 times, most recently from 14e26f1 to 1f57332 Compare May 3, 2024 03:22
@@ -258,7 +258,7 @@
"tooltip": {
"timeRange": {
"title": "Time Range",
"text": "This dropdown menu allows you to pick the time range for the data you want to inspect. Note that the dashboard limits the data to 60 days."
"text": "This graph displays the historical trend of signal strength in your chosen location for the time period selected."
Copy link
Contributor Author

Choose a reason for hiding this comment

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

How do the other language app.json files get updated?

@moconnell9
Copy link
Contributor Author

@salcock Can you please help me deploy this to https://ioda-ux.inetintel.cc.gatech.edu/ when it is ready to be merged?

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.

1 participant