RSS feeds should not be in head for mobile
Closed, ResolvedPublic
Actions

Description

There are 4 atom feeds in the head of mobile.
<link rel="alternate" type="application/atom+xml" title="Wikipedia Atom feed" href="/w/index.php?title=Special:RecentChanges&feed=atom">
These are arguably useless on mobile.

It should be possible for a skin to turn these off if necessary.

Details

	Subject	Repo	Branch	Lines +/-
	RSS feeds should not be in head for mobile	mediawiki/extensions/MobileFrontend	master	+12 -1

Customize query in gerrit

Related Objects
Search...

Status	Assigned	Task
Duplicate	• Jhernandez	T104432 [EPIC]: Improve mobile site performance
Resolved	Jdlrobson	T98986 [EPIC] Improve the status quo of mobile web performance
Resolved	Albert221	T100377 RSS feeds should not be in head for mobile

Event Timeline

Jdlrobson created this task.May 25 2015, 10:01 PM

Jdlrobson raised the priority of this task from to Needs Triage.

Jdlrobson updated the task description. (Show Details)

Jdlrobson added projects: Epic, Performance Issue, Web-Team-Backlog.

Jdlrobson added subscribers: • Mholloway, Ricordisamoa, He7d3r and 7 others.

phuedx removed a project: Epic.May 28 2015, 1:53 PM

phuedx set Security to None.

phuedx moved this task from Incoming to 2014-15 Q4 on the Web-Team-Backlog board.

@Jdlrobson does this have any impact on performance? (besides html size)

How would we go about working on this?

• Jhernandez triaged this task as Medium priority.Aug 17 2015, 11:19 AM

• Jhernandez edited projects, added MobileFrontend; removed Web-Team-Backlog.Aug 17 2015, 11:25 AM

Jdlrobson mentioned this in T105365: Make a performance audit of the mobile site.Aug 27 2015, 11:17 PM

• bmansurov added a project: Google-Code-In-2015.Nov 30 2015, 12:33 PM

It's possible in Safari on iOS to subscribe to RSS feeds (with the shared links feature), so they are not completely useless.

https://codein.withgoogle.com/tasks/4908067837706240/

Aklapper moved this task from Proposed to Imported into GCI site on the Google-Code-In-2015 board.Dec 1 2015, 2:04 PM

@Aklapper Your link gives me a 403.

• Mholloway unsubscribed.Dec 1 2015, 7:27 PM

Jdlrobson lowered the priority of this task from Medium to Lowest.Dec 3 2015, 11:40 PM

In T100377#1842290, @Volker_E wrote:

@Aklapper Your link gives me a 403.

The page might require login/registration for GCI, not sure...
@Volker_E: Would you like to (co-)mentor, or have some potential GCI tasks in mind yourself? If so I'm very happy to send you a registration email!

In T100377#1838030, @Sjoerddebruin wrote:

It's possible in Safari on iOS to subscribe to RSS feeds (with the shared links feature), so they are not completely useless.

Hmm, this is a valid concern, I think. How do we want to deal with it? Removing the rss feed link nonetheless?

These are the links in head I see on a normal page view:

<link rel="alternate" type="application/x-wiki" title="Bearbeiten" href="/w/index.php?title=TestSections&amp;action=edit">
<link rel="edit" title="Bearbeiten" href="/w/index.php?title=TestSections&amp;action=edit">
<link rel="shortcut icon" href="/favicon.ico">
<link rel="search" type="application/opensearchdescription+xml" href="/w/opensearch_desc.php" title="devwiki (en)">
<link rel="EditURI" type="application/rsd+xml" href="http://localhost/w/api.php?action=rsd">
<link rel="copyright" href="//creativecommons.org/licenses/by-sa/3.0/">
<link rel="alternate" type="application/atom+xml" title="Atom-Feed für „devwiki“" href="/w/index.php?title=Special:RecentChanges&amp;feed=atom">

As far as I can see, there is only one RSS feed link. However, there are other alternate links, too, is this meant as "4 rss links"? :) Btw.: @Albert221 asked me in IRC, and I said, that it would be ok to omit the one rss feed link to complete the GCI task :)

• Jhernandez unsubscribed.Dec 16 2015, 8:31 AM

Change 259546 had a related patch set uploaded (by Albert221):
RSS feeds should not be in head for mobile

https://gerrit.wikimedia.org/r/259546

gerritbot added a project: Patch-For-Review.Dec 16 2015, 7:48 PM

Albert221 mentioned this in T121774: Find better way to omit outputting feed links.Dec 17 2015, 4:58 PM

Change 259546 merged by jenkins-bot:
RSS feeds should not be in head for mobile

https://gerrit.wikimedia.org/r/259546

ReleaseTaggerBot added a project: MW-1.27-release (WMF-deploy-2016-01-12_(1.27.0-wmf.10)).Dec 17 2015, 10:00 PM

Florian closed this task as Resolved.Dec 17 2015, 10:06 PM