Make WordPress Core

Opened 7 years ago

Closed 4 years ago

Last modified 4 years ago

#49099 closed enhancement (fixed)

Twenty Twenty: Add social icon for Mastodon

Reported by: rryyaanndd Owned by: audrasjb
Priority: normal Milestone: 6.2
Component: Bundled Theme Version:
Severity: normal Keywords:
Cc: Focuses:

Description (last modified by sabernhardt)

Please add a social icon for ​Mastodon in the Twenty Twenty Theme.

Attachments (1)

Capture d’écran 2020-01-11 à 17.47.28.png​ (51.1 KB ) - added by audrasjb 7 years ago.
Mastodon icon on Twenty twenty

Download all attachments as: .zip

Change History (9)

#1 @audrasjb
7 years ago

  • Keywords reporter-feedback close added

Hi and thank you for opening this ticket,
Mastodon icon is already available, but it's only triggered if the provided URL contains mastodon.com URL.

@audrasjb
7 years ago

Mastodon icon on Twenty twenty

#2 @ianbelanger
7 years ago

  • Milestone Awaiting Review
  • Resolution → worksforme
  • Status new → closed

Closing as a worksforme

#3 @rryyaanndd
7 years ago

  • Resolution worksforme
  • Status closed → reopened

The problem is mastodon.com isn't a real domain name for mastodon instances. So nobody will use that icon since there will never be a link to that domain.
There are many different domains for mastodon instances (thousands):

​https://joinmastodon.org/#getting-started
​https://instances.social/list/advanced

#4 @ianbelanger
7 years ago

  • Resolution → wontfix
  • Status reopened → closed

Hi @rryyaanndd,

Because of the way the social icon navigation works, using the url to trigger the correct icon and since there are thousands of domain instances for mastodon, I am not sure that we can add mastodon as an option at this point. If you can provide a solution that works, please submit a patch here or create a plugin. At this point though I am closing this ticket as a wontfix.

#5 @sabernhardt
4 years ago

  • Description modified (diff)
  • Keywords reporter-feedback close removed
  • Milestone → 6.2
  • Resolution wontfix
  • Status closed → reopened

We should map at least the original Mastodon domain, mastodon.social, so the theme does not check mastodon.com. Discussion about adding a small selection of other possibilities is on #57293.

#6 @audrasjb
4 years ago

  • Owner set to audrasjb
  • Resolution → fixed
  • Status reopened → closed

In 55061:

Bundled Themes: Add Mastodon domains for menu item icons.

This changeset adds the top 10 Mastodon domains by popularity to Twenty Twenty and Twenty Twenty-One Social icon feature.

Note: other Mastodon domains can be added to the array of domain mapping using twentytwenty_social_icons_map and twenty_twenty_one_social_icons_map filters.

Props triumvirate, sabernhardt, audrasjb, peterwilsoncc, rryyaanndd, ianbelanger.
Fixes #57293, #49099.

#7 @milana_cap
4 years ago

  • Keywords needs-user-docs added

#8 @milana_cap
4 years ago

  • Keywords needs-user-docs removed
Note: See TracTickets for help on using tickets.