Making WordPress.org

Opened 6 years ago

Closed 5 years ago

Last modified 5 years ago

#4124 closed defect (bug) (fixed)

Update Slack logo

Reported by: tobifjellner's profile tobifjellner Owned by: dd32's profile dd32
Milestone: Priority: normal
Component: General Keywords:
Cc:

Description

In January 2019 Slack updated their logo.
Their current logos are available at https://brandfolder.com/slack/logos

So it might be good to update various places that show the Slack logo in meta:

And probably more places.

Attachments (4)

4124.chat-banner.png (43.7 KB) - added by dufresnesteven 5 years ago.
Update to the chat image.
4124.diff (4.7 KB) - added by dufresnesteven 5 years ago.
Change Base 64 in style.css to new Slack logo.
Slack_Mark-96x96-abe6ce9.png (3.0 KB) - added by dufresnesteven 5 years ago.
Image for trac.
4124.chat-mobile.png (124.9 KB) - added by dufresnesteven 5 years ago.
Mobile banner for /chat

Download all attachments as: .zip

Change History (13)

This ticket was mentioned in Slack in #forums by otto42. View the logs.


6 years ago

@dufresnesteven
5 years ago

Update to the chat image.

@dufresnesteven
5 years ago

Change Base 64 in style.css to new Slack logo.

@dufresnesteven
5 years ago

Image for trac.

#2 @dufresnesteven
5 years ago

I don't have access to make these image changes.

Can someone with access get this over the finish line?

Last edited 5 years ago by dufresnesteven (previous) (diff)

#3 follow-up: @dd32
5 years ago

Updated the header image on https://make.wordpress.org/chat/ - Looks like the Mobile App logo needs updating too

#4 in reply to: ↑ 3 ; follow-up: @dufresnesteven
5 years ago

Do these changes not required access to production? I don't see these in a template in the code base.

Url: https://make.wordpress.org/chat/files/2019/12/cropped-4124.chat-banner.png

Replying to dd32:

Updated the header image on https://make.wordpress.org/chat/ - Looks like the Mobile App logo needs updating too

#5 in reply to: ↑ 4 @dd32
5 years ago

Replying to dufresnesteven:

Do these changes not required access to production? I don't see these in a template in the code base.

Some pages that are english-only are just WordPress pages (which can be a surprise sometimes..) in this case it was a Header image, so no code changes required for that part.

@dufresnesteven
5 years ago

Mobile banner for /chat

#6 @dufresnesteven
5 years ago

Currently Embedded Into Page:

a[href^="https://wordpress.slack.com/"] {
    padding-left: 20px;
    background: url(https://slack.global.ssl.fastly.net/20655/img/icons/favicon-32.png) no-repeat;
    background-size: 16px 16px;
}

Needs to be updated to:

a[href^="https://wordpress.slack.com/"] {
    padding-left: 1.5rem;
    background: url(data:image/svg+xml;base64,PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0idXRmLTgiP…45LDEyLjljMCw3LjEtNS44LDEyLjktMTIuOSwxMi45SDE1MS4yeiIvPjwvZz48L2c+PC9zdmc+) no-repeat;
    background-size: 2.25rem;
    background-position: -0.35rem -0.25rem;
}

#7 @dd32
5 years ago

Updated Trac image. It'll take a bit to flow through, since it's a gravatar attached to a certain someones email (I think it's only cached for 5 mins, but there's also our redirect).

Looks like the Mobile App logo needs updating too

Thanks, updated that one too now. It's just a image block in page content.

#8 @dd32
5 years ago

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

In 9332:

Breathe: Update the slack logo.

Props dufresnesteven.
Fixes #4124.

#9 @dd32
5 years ago

In 9333:

Breathe: Cache bust after r9332.

See #4124.

Note: See TracTickets for help on using tickets.