Twenty Fifteen: Show RSS icon instead of WordPress icon in Social Links Menu

[artychan]

Would really appreciate it if adding a feed link (i.e. username.wordpress.com/feed ) to the Social Links Menu shows the RSS icon instead of the WordPress icon.

I would expect something similar to what's on the Social Links documentation:

Right now, I'm having to use the separate RSS Links widget. While not difficult to do, it means making my menu significantly longer (because of all the extra padding and such) to add this one link.

[SergeyBiryukov]

The RSS icon is already there​, but it currently expects a trailing slash.

I guess we could add another rule for /feed.

[artychan]

Yes, please. A user would never realize that a trailing slash is necessary.

Even with trailing slash, it's not working for me though. Using the hosted WordPress site.

In the menu, this is what I have:
Custom Link
URL: ​http://cynng.wordpress.com/feed/
Navigation Label: RSS

However, I saved it, and then on my blog, it's still showing the WordPress icon. WordPress automatically changes it to the secure version (on the public side, dashboard setting still shows regular http), ​https://cynng.wordpress.com/feed/ Not sure if that's affecting it?

You can see it in action on my blog, first social links icon: ​http://cynng.wordpress.com

[SergeyBiryukov]

It looks like Twenty Fifteen on WordPress.com has some additional rules for wordpress.org and wordpress.com, which override the RSS icon rule.

If I disable those additional rules, I can see the RSS icon (see the screenshot​).

[SergeyBiryukov]

Replying to SergeyBiryukov:

It looks like Twenty Fifteen on WordPress.com has some additional rules for wordpress.org and wordpress.com, which override the RSS icon rule.

Correction: The standalone version of Twenty Fifteen has these rules too.

We should probably move the RSS icon rule to the bottom, to prevent it from being overridden by those rules.

[iamtakashi]

Replying to artychan:

Yes, please. A user would never realize that a trailing slash is necessary.

From readme.txt

RSS Feed (URLs with /feed/)

But thanks for reporthing this.

Replying to SergeyBiryukov:

We should probably move the RSS icon rule to the bottom, to prevent it from being overridden by those rules.

Good point. Let's do that.

[iamtakashi]

Move RSS icon style rule at the bottom to prevent it from being overridden by other social icon rules.

[lancewillett]

In 31283:

Twenty Fifteen: move RSS icon style rule lower to prevent it from being overridden by other social icon rules.

Props iamtakashi, fixes #31129.

[artychan]

Big thanks everyone, especially @iamtakashi for patch.