Last modified: 2014-06-18 10:37:46 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T57917, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 55917 - apple-touch-icon.png, $wgAppleTouchIcon and docroot cleanup
apple-touch-icon.png, $wgAppleTouchIcon and docroot cleanup
Status: NEW
Product: Wikimedia
Classification: Unclassified
General/Unknown (Other open bugs)
wmf-deployment
All All
: Normal enhancement (vote)
: ---
Assigned To: Sam Reed (reedy)
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-10-19 18:18 UTC by Sam Reed (reedy)
Modified: 2014-06-18 10:37 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Sam Reed (reedy) 2013-10-19 18:18:45 UTC
[18:28:26] <Reedy> Anyone familiar with the apple-touch-icon?
[18:30:08] <Reedy> Do apple products check for the existence of domain.ext/apple-touch-icon.png? Or do they use the <link rel="apple-touch-icon" href=" ?
[18:30:38] <Reedy> We've one wiki (enwiktionary) that is setup to use $wgAppleTouchIcon and hence output the link above in <head>
[18:30:56] <Reedy> And a hanful of others with apple-touch-icon.png images in their docroots (not in /w/)
[18:32:22] <Reedy> Halp.
Comment 1 Kunal Mehta (Legoktm) 2013-10-19 20:30:49 UTC
According to https://developer.apple.com/library/ios/documentation/AppleApplications/Reference/SafariWebContent/ConfiguringWebApplications/ConfiguringWebApplications.html both work, and <link...> will override the one at domain.ext/apple-touch-icon.png
Comment 2 Sam Reed (reedy) 2013-10-20 01:11:50 UTC
So I'm likely to move most of those from their respective docroots onto bits or similar (like enwiktionary is currently), changing most of them to then be a link, rather than physical files in random docroots. Normalising the configuration somewhat.

I'm wondering if we need to do something like we do for favico for the apple touch images; rewrite /fav.ico to a php script, which pulls it from $wgFavIco, or we can just get by with it being as a page link.
Comment 3 Sam Reed (reedy) 2013-10-20 01:23:07 UTC
'wgAppleTouchIcon' => array(
	'default' => false, // iOS searches for icons in docroot by default
	'enwiktionary' => '//bits.wikimedia.org/apple-touch/wiktionary/en.png', // bug 46431
),

The "by default" comment leans towards we should have a rewrite and script for it..
Comment 4 Brion Vibber 2013-10-20 12:01:28 UTC
I don't think a redirector is necessary if we set the default value, but if it's easier to manage than the config file that should work fine too. :)
Comment 5 Gerrit Notification Bot 2013-10-20 16:47:25 UTC
Change 90762 had a related patch set uploaded by Reedy:
Moved all apple-touch-icon.png images to bits. Set $wgAppleTouchIcon where appropriate.

https://gerrit.wikimedia.org/r/90762
Comment 6 Sam Reed (reedy) 2013-10-20 16:51:05 UTC
(In reply to comment #5)
> Change 90762 had a related patch set uploaded by Reedy:
> Moved all apple-touch-icon.png images to bits. Set $wgAppleTouchIcon where
> appropriate.
> 
> https://gerrit.wikimedia.org/r/90762

This gives essentially the same behaviour, but makes all the files be handled in the same way. So it can be merged without changing any of the current behaviour.

What should we do for default? I know we tend to default to enwiki for somethings... Wikimedia might be more sensible.

Addition of more project images, and/or a default would be nice to be done in the near future.

This helps with my cleanup of most of the redundant docroot folders we have (removing apple touch just leaves the 3 standard symlinks)
Comment 7 Gerrit Notification Bot 2013-10-21 09:14:46 UTC
Change 90762 merged by jenkins-bot:
Moved all apple-touch-icon.png images to bits. Set $wgAppleTouchIcon where appropriate.

https://gerrit.wikimedia.org/r/90762
Comment 8 Sam Reed (reedy) 2013-10-21 10:16:02 UTC
https://bugzilla.wikimedia.org/show_bug.cgi?id=19392#c15

https://gerrit.wikimedia.org/r/#/c/60777/


It seems this might cause a problem for older apple devices... It shouldn't be too much work (copy and amend) to modify the favicon script to do something similar for a different global and to a different file...

I've deployed my change above, but not propagated the deletion (sync-docroot doesn't delete)
Comment 9 Tomasz W. Kozlowski 2013-10-21 10:22:38 UTC
CC-ing Krinkle, who was involved in a discussion around 60777 (and who, I think, also suggested re-using the favicon script for Apple Touch icons).
Comment 10 Andre Klapper 2014-03-11 16:53:27 UTC
No patches to review left here; resetting status

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links