Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add sync icon #5296

Merged
merged 1 commit into from
Jul 24, 2021
Merged

Add sync icon #5296

merged 1 commit into from
Jul 24, 2021

Conversation

ByteHamster
Copy link
Member

I did a user test yesterday and got the feedback that a star for the synchronization icon is weird. Replaced it with a cloud icon.

cc @thrillfall

@ByteHamster ByteHamster merged commit 210e326 into AntennaPod:develop Jul 24, 2021
@ByteHamster ByteHamster deleted the sync-icon branch July 24, 2021 15:22
@thrillfall
Copy link
Contributor

I don't get why its a cloud. We synchronize the data with very specific services (gpodder.net, nextcloud)
This is synchronization and not cloud computing or hosting with aws/...
The sync symbol (2 circling arrows) represents exactly that

@ByteHamster
Copy link
Member Author

I fear that using cycling arrows makes it less clear to users if AntennaPod is currently synchronizing with nextcloud or if it is refreshing the podcast subscriptions. Both would use circling arrows. Nextcloud even has the word "cloud" in its name, so I think it should be okay.

@keunes
Copy link
Member

keunes commented Jul 24, 2021

Cloud is not just AWS. I consider my NC instance my private cloud - an always-on always-connected & publicly accessible computer.

And synchronisation in most cases happens via the www (rather than LAN). I also feel that a cloud is also an oft-used symbol for synchronisation, so for an icon in the settings it's well-suited.

That said, I agree that the arrows could nicely represent synchronisation. But all in all for the settings the cloud is better than having horizontal and vertical arrows just used above each-other (the setting 'Network' uses the vertical arrows).

@antennapod-bot
Copy link

This pull request has been mentioned on AntennaPod Forum. There might be relevant details there:

https://forum.antennapod.org/t/antennapod-2-4-release-notes/1300/1

@antennapod-bot
Copy link

This pull request has been mentioned on AntennaPod Forum. There might be relevant details there:

https://forum.antennapod.org/t/antennapod-2-4-release-notes/1300/3

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.

4 participants