WordPress.org
Get WordPress

Making WordPress.org

Opened 9 years ago

Closed 9 years ago

#1283 closed defect (bug) (fixed)

BrowseHappy.com - Add site Icon

Reported by: ramiy's profile ramiy Owned by: coffee2code's profile coffee2code
Milestone: Priority: normal
Component: Browse Happy Keywords: has-patch
Cc:

Description

Currently, BrowseHappy.com has 3 logo images on it's theme files.
Source: https://meta.trac.wordpress.org/browser/sites/trunk/browsehappy.com/public_html/imgs

And the theme has hardcoded the <link> and <meta> tags for the favicon and apple-touch-icon.
Source: https://meta.trac.wordpress.org/browser/sites/trunk/browsehappy.com/public_html/index.php#L15

As of WP 4.3, this functionality is built into WordPress core. We just need someone with permission to add the site icon. And commit the attached patch.

Attachments (1)

meta-1283.patch​ (1.1 KB) - added by ramiy 9 years ago.

Download all attachments as: .zip

Change History (9)

@ramiy
9 years ago

#1 @ramiy
9 years ago

Commit this patch after adding "Site Icon".

And delete the 3 apple-touch icons in the theme folder.

#2 @ramiy
9 years ago

Related: #85 New Favicon for jobs.wordpress.net

#4 @ramiy
9 years ago

@melchoyce, do you know who has access to BrowseHappy.com admin dashboard ?

#5 @melchoyce
9 years ago

I don't, unfortunately. Maybe someone on the meta team?

This ticket was mentioned in ​Slack in #meta by drew. ​View the logs.
9 years ago

#7 @Otto42
9 years ago

I'm working on gaining access to it now. coffee2code should be able to update my key on that system.

Version 0, edited 9 years ago by Otto42 (next)

#8 @coffee2code
9 years ago

  • Owner set to coffee2code
  • Resolution set to fixed
  • Status changed from new to closed

In 2732:

Browse Happy: Utilize site icon feature for site icons.

Props ramiy.
Fixes #1283.

Note: See TracTickets for help on using tickets.