Skip to content

icons: Switch to conventional track skip icons#252

Merged
jpochyla merged 1 commit intojpochyla:masterfrom
jforberg:icons
Mar 19, 2022
Merged

icons: Switch to conventional track skip icons#252
jpochyla merged 1 commit intojpochyla:masterfrom
jforberg:icons

Conversation

@jforberg
Copy link
Copy Markdown
Contributor

@jforberg jforberg commented Jan 8, 2022

Change the track skipping buttons from ⏩, ⏪ to ⏭, ⏮. The shapes are
from the SF Pro font, same as the old icons.

The icons ⏩, ⏪ conventionally denote fast-forward and rewind (seeking
within a single track). By contrast, the icons ⏭, ⏮ denote track
skipping (moving to the next or previous track).

See for instance: https://en.wikipedia.org/wiki/Media_control_symbols#Symbols

@jacksongoode
Copy link
Copy Markdown
Collaborator

Could you post a screenshot of the new svgs?

@jforberg
Copy link
Copy Markdown
Contributor Author

jforberg commented Feb 5, 2022

2022-02-05-105347_1196x152_scrot

@jpochyla
Copy link
Copy Markdown
Owner

Hi @jforberg, thanks for the contribution! Can you please fill the names of the icons in the comment on top of the respective const? Merging afterwards.

Change the track skipping buttons from ⏩, ⏪ to ⏭, ⏮. The shapes are
from the SF Pro font, same as the old icons.

The icons ⏩, ⏪ conventionally denote fast-forward and rewind (seeking
within a single track). By contrast, the icons ⏭, ⏮ denote track
skipping (moving to the next or previous track).

See for instance: https://en.wikipedia.org/wiki/Media_control_symbols#Symbols
@jforberg
Copy link
Copy Markdown
Contributor Author

Hi @jpochyla, I updated the icon names as requested

@jpochyla jpochyla merged commit 2d2cf65 into jpochyla:master Mar 19, 2022
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.

3 participants