Page MenuHomePhabricator

Update favicon wiktionary/en.ico
Closed, ResolvedPublic

Description

Author: adityagulati

Description:
32x32 size

Create a favicon including 16x16, 32x32, and 48x48 versions of https://upload.wikimedia.org/wikipedia/commons/3/35/Simple_Wiktionary_logo.png. Substitute wiktionary/en.ico at operations/mediawiki-config/docroot/bits/favicon with the new version.


Version: wmf-deployment
Severity: enhancement
URL: http://www.google-melange.com/gci/task/view/google/gci2013/5777530279690240

Attached:

Details

Reference
bz58326

Related Objects

StatusSubtypeAssignedTask
ResolvedNone
ResolvedSn1per

Event Timeline

bzimport raised the priority of this task from to Low.Nov 22 2014, 2:42 AM
bzimport set Reference to bz58326.

adityagulati wrote:

16x16 size

Attached:

adityagulati wrote:

48x48 size

Attached:

Hello Aditya, please follow the instructions of the Google Code-in task.

What we are looking for is one single favicon with three layers contributed as a Gerrit changeset.

As an example, check Bug 58144 and its related https://gerrit.wikimedia.org/r/100133

You have links with information about Git and Gerrit at https://www.mediawiki.org/wiki/Google_Code-In

Thank you!

Given comment 3, I'm presuming that this bug is not unconfirmed.

  • Bug 58523 has been marked as a duplicate of this bug. ***

Hi Adit,

(In reply to bug 58523 comment #2)

Since I have the .ico made, should I just make a pull request to the main
public repo so that it will be uploaded to the main project?

You must submit a patch via Gerrit, specifying in the commit message the bug report you are solving. See for example bug 58122 and its related https://gerrit.wikimedia.org/r/#/c/99756/

A pull request in GitHub is not bad for a first try but we are expecting the patch to follow the Gerrit review process that we apply to all our contributions. If you are not familiar with Gerrit you can check the tutorial and other interesting links at

https://www.mediawiki.org/wiki/Google_Code-In#Suggested_reading

We hope to see the link to your patch for review soon!

Change 102386 had a related patch set uploaded by Aquifacae:
Changed wiktionary/en.ico favicon to proper image and resolutions

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

Change 102386 abandoned by Aquifacae:
Changed wiktionary/en.ico favicon to proper image and resolutions

Reason:
Decided not to use this image for favicon

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

Change 102554 had a related patch set uploaded by Aquifacae:
Added new 48x48 resolution to original favicon

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

Change 102554 abandoned by Aquifacae:
Added new 48x48 resolution to wiktionary/en favicon

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

Change 102602 had a related patch set uploaded by Aquifacae:
Made proper larger favicon wiktionary/en.ico favicon.

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

Change 102602 abandoned by Aquifacae:
Made proper larger favicon wiktionary/en.ico.

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

Change 103183 had a related patch set uploaded by Sn1per:
Fix en.wiktionary favicon for GCI 2013

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

Change 103183 merged by jenkins-bot:
Fix en.wiktionary favicon

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