Skip to content

Make the liked icon configurable#659

Merged
Rigellute merged 4 commits into
Rigellute:masterfrom
Utagai:configurable-liked-icon
Nov 24, 2020
Merged

Make the liked icon configurable#659
Rigellute merged 4 commits into
Rigellute:masterfrom
Utagai:configurable-liked-icon

Conversation

@Utagai
Copy link
Copy Markdown
Contributor

@Utagai Utagai commented Nov 14, 2020

This closes #625.

Sorry this took longer than it should have. I got distracted.

Anyways, it's a pretty simple change. I only have one reservation about the naming.

Here's a screenshot of it in action:
image

And here's the accompanying config:

theme:
  ...

behavior:
  liked_icon: ""

@Utagai Utagai changed the title Configurable liked icon Make the liked icon configurable Nov 14, 2020
Comment thread README.md
show_loading_indicator: true
# Determines the text icon to display next to "liked" Spotify items, such as
# liked songs and albums, or followed artists. Can be any length string.
liked_icon: "♥"
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

My reservation is here. Technically, in almost all cases this is indeed the "liked icon". However, for artists, one "follows" them, so it kind of breaks there.

I considered some alternative names like "tracked_icon" or similar, but it just didn't feel right. Open to any renaming ideas.

@OrangeFran
Copy link
Copy Markdown
Contributor

Hey.

I really like this. Thank you. Would you like to implement this for other icons, like shuffle, repeat and currently playing. I'm working on a cli and using icons a lot so I'm gonna implement that.

@Utagai
Copy link
Copy Markdown
Contributor Author

Utagai commented Nov 14, 2020

@OrangeFran It's unclear from your message, are you asking if I'd be interested in implementing custom icons for other things besides this, or are you saying you're going to be doing that?

@OrangeFran
Copy link
Copy Markdown
Contributor

Sorry, I meant that I'm gonna change from static icons to the configured ones in my fork of the project after this PR gets merged. I wanted to ask if you would be interested in doing the same for other icons. Hopefully this cleares things up.

@Utagai
Copy link
Copy Markdown
Contributor Author

Utagai commented Nov 14, 2020

I think I get it. Yeah, I was thinking of doing it for the current arrow marker as well. Not aware of other icons though. Granted, I've never tried shuffle. You mention for example, repeat, but there is no repeat text icon. Does that mean you're suggesting we add one? For example, Repeat currently just uses a string, e.g., "All", "Track", etc.

@OrangeFran
Copy link
Copy Markdown
Contributor

OrangeFran commented Nov 14, 2020

Yeah, adding a repeat icon would be cool. But there probably should be a seperate PR for that. I thought they were already there, my bad, because I'm working on feature myself where I use repeat and shuffle icons and thought they were already implemented in the code.

@Utagai
Copy link
Copy Markdown
Contributor Author

Utagai commented Nov 14, 2020

No worries. But yes, there really aren't any other icons besides the one made configurable in this PR and the current marker (arrow). We could perhaps add configurable ones for repeat/shuffle, but I'm unsure of what valid options there could be. For example, a simple search for "repeat" and "loop" over the unicode characters do not reveal many sensible options. Especially considering that repeat has various modes, it becomes even harder to distinguish them.

Did you have particular icons in mind?

@OrangeFran
Copy link
Copy Markdown
Contributor

True, I thought of nerd font icons. They have pretty nice shuffle and repeat icons. But there are no unicode icons, at least I didn't find any. Maybe that's the reason that there aren't any more icons implemented.

@Utagai
Copy link
Copy Markdown
Contributor Author

Utagai commented Nov 14, 2020

Yeah, nerd font, font awesome, etc, could open the doors to something that applies.

In either case, definitely doable. This PR shows it shouldn't be too bad to extend to a few more possibilities. I'm open to doing it if there's interest and @Rigellute is OK with it.

Copy link
Copy Markdown
Owner

@Rigellute Rigellute left a comment

Choose a reason for hiding this comment

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

Great work @Utagai, this looks like a nice addition. Thank you 🙏

@Rigellute Rigellute merged commit 6f01c09 into Rigellute:master Nov 24, 2020
lanej pushed a commit to lanej/spotify-tui that referenced this pull request Jul 13, 2021
* Add liked icon to config and use it over the default

* Add padding to the liked icon

* Add new liked_icon option to README.md

* Fix issue with double-padding
nighi pushed a commit to nighi/spotify-tui that referenced this pull request Jul 11, 2025
* Add liked icon to config and use it over the default

* Add padding to the liked icon

* Add new liked_icon option to README.md

* Fix issue with double-padding
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.

Make the liked icon configurable

3 participants