Skip to content

added titles for icons throughout site#2263

Open
venunalla7731 wants to merge 4 commits intocovid19india:masterfrom
venunalla7731:master
Open

added titles for icons throughout site#2263
venunalla7731 wants to merge 4 commits intocovid19india:masterfrom
venunalla7731:master

Conversation

@venunalla7731
Copy link
Copy Markdown

@venunalla7731 venunalla7731 commented Jul 22, 2020

Description of PR
This is related to the information to show when I am mouse over on small icons on the web page(screenshot added below).
In order to achieve showing info on mouse-over, added the title to icons. like anchor icon next to "Spread Trends" and etc., which users might wonder what it does?
Made the changes accordingly.

Checklist

  • Compiles and passes lint tests
  • Properly formatted
  • Tested on desktop
  • Tested on phone

Screenshots

image

image

image

image

image

@venunalla7731
Copy link
Copy Markdown
Author

Can anyone let me know anything that needs to add in order to get approved this PR?
@JunaidBabu @KhanStan99

@venunalla7731
Copy link
Copy Markdown
Author

@shuklaayush Can you please look into this, and let me know if any required?

@venunalla7731
Copy link
Copy Markdown
Author

@jeremyphilemon @shuklaayush please approve this PR or let me know if any changes required.

@venunalla7731
Copy link
Copy Markdown
Author

HI, @jeremyphilemon can move this PR to complete?

@venunalla7731
Copy link
Copy Markdown
Author

Hi all,
Do we have any update on this PR?

>
<div className="timeseries-header">
<div
title="Hepls you to toggle with/without maps."
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Misspelled - Helps

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.

2 participants