Add tooltips to Clear & Settings buttons #1021

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants
@JanCVanB

JanCVanB commented Apr 6, 2017

Add the same type of HTML titles (and aria-labels) as on other icon buttons.

JanCVanB and others added some commits Apr 6, 2017

Add tooltips to Clear & Settings buttons
Add the same type of HTML titles (and aria-labels) as on other icon buttons.
@yiskah

yiskah approved these changes Apr 8, 2017

Looks good to me. I think someone else should check this too though

@Gargron

This comment has been minimized.

Show comment
Hide comment
@Gargron

Gargron Apr 8, 2017

Member

I believe this has been already done in #1101 with localizations, sorry! Thank you for your effort. The only thing the other PR was missing was aria-label, so if you wanna reopen your PR just to add those, you are welcome to!

Member

Gargron commented Apr 8, 2017

I believe this has been already done in #1101 with localizations, sorry! Thank you for your effort. The only thing the other PR was missing was aria-label, so if you wanna reopen your PR just to add those, you are welcome to!

@Gargron Gargron closed this Apr 8, 2017

abcang added a commit to pixiv/mastodon that referenced this pull request Apr 17, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment