Add support for remote favicons

PleromaFE: pleroma-fe!1076 (merged)

changed the description

mentioned in merge request pleroma-fe!1076 (merged)

added 1 commit

• 39526b62 - Add support for remote favicons

Compare with previous version

added 1 commit

• 90659bf1 - Add support for remote favicons

Compare with previous version

added 1 commit

• 55613b2e - Move get_favicon to Pleroma.Instances, use /

Compare with previous version

added 1 commit

• 7853da68 - Move get_favicon to Pleroma.Instances, use /

Compare with previous version

added 1 commit

• b2c8c0f6 - instances.ex: Remove comment about using /

Compare with previous version

unmarked as a Work In Progress

I kind of want this to be optional and off-by-default, at least in the current state.

Reactivating discussion on this, seems like a fun idea but why not make it more proper and add a 'favicon' column to our instances table and pull these in from time to time?

Just wanted to chime in that I think this is a very nice idea. "Visually distinguish local from remote users" is something requested in Soapbox a long time ago because it helps newcomers to the fediverse understand how it works.

Without this, we would see users saying things like "Why are there Nazis on this site??" while clearly referring to a Gab user, and not understanding federation.

Qoto.org also has something like this, for example:

I eventually added a disclaimer to the fediverse tab to try to make it more clear:

But even still, people would actually forget they are in the fediverse tab.

I think the favicon would serve as a great visual indicator for newcomers, almost like a tutorial.

I think in Soapbox FE I want clicking the favicon to go to the remote server's timeline: !2713 (merged)

Just a comment so I don't forget.

added 2042 commits

• b2c8c0f6...69f0b286 - 2039 commits from branch develop
• 1a7a5d31 - Add support for remote favicons
• bf07c9b9 - Move get_favicon to Pleroma.Instances, use /
• 5baa0e71 - Use instances table instead of Cachex

Compare with previous version

added 1 commit

• d18d3292 - instance: Prevent loop of updates

Compare with previous version

added 2 commits

• b332d8d2 - Use instances table instead of Cachex
• b81fee4d - instance: Prevent loop of updates

Compare with previous version

added 1 commit

• 6c8b7a78 - Add [:instances_favicons, :enabled] setting, defaults to false

Compare with previous version

resolved all threads

@lambadalambda Now, disabled by default and using the instances table. (Also live on my instance)

added 2 commits

• c09bbb75 - Add [:instances_favicons, :enabled] setting, defaults to false
• 3bc8dde3 - Add changelog and documentation

Compare with previous version

added 54 commits

• 3bc8dde3...a8447c38 - 48 commits from branch develop
• f6d09faf - Add support for remote favicons
• 6a679d80 - Move get_favicon to Pleroma.Instances, use /
• 013e2c50 - Use instances table instead of Cachex
• 8c9df2d2 - instance: Prevent loop of updates
• 312fc55f - Add [:instances_favicons, :enabled] setting, defaults to false
• 31fef95e - Add changelog and documentation

Compare with previous version