Skip to content

Three small changes: play/pause tooltip, mat icon typo, aria recommendation#220

Merged
joanise merged 4 commits intomainfrom
dev.ej-misc
Jul 6, 2023
Merged

Three small changes: play/pause tooltip, mat icon typo, aria recommendation#220
joanise merged 4 commits intomainfrom
dev.ej-misc

Conversation

@joanise
Copy link
Copy Markdown
Member

@joanise joanise commented Jul 6, 2023

No description provided.

@joanise joanise requested a review from dhdaines July 6, 2023 14:59
@joanise
Copy link
Copy Markdown
Member Author

joanise commented Jul 6, 2023

Note: I'm not sure my Spanish translation is OK, with "Déle play/pause", but I was adapting from the existing text. I'm hoping for a native speaker review before merging.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 6, 2023

PR Preview Action v1.4.4
🚀 Deployed preview to https://ReadAlongs.github.io/Web-Component/pr-preview/pr-220/
on branch gh-pages at 2023-07-06 16:44 UTC

Copy link
Copy Markdown
Collaborator

@dhdaines dhdaines left a comment

Choose a reason for hiding this comment

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

I will defer to a native speaker for the Spanish translation (which I find a bit shocking! where's the equivalent of the OQLF?) but aside from that this seems quite sensible.

with native speaker review on Reproducir/pausar

and make the tooltips that are next to each other consistent
@joanise joanise merged commit 38de8db into main Jul 6, 2023
@joanise joanise deleted the dev.ej-misc branch July 6, 2023 17:03
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