search

rackerlabs / design-system

Design documentation for Helix, the Rackspace Design System
11 stars 14 forks source link

feat(logos): add CDN logos to iconography page #464

Closed technabors closed 6 years ago

technabors commented 6 years ago

This PR adds a new <section> for logos on the iconography page. Additionally a new class was generated to adjust the background color of the <figure> element for the light logo to display. #333333 was used as the bg-color, however this can be adjusted as needed.

I also wonder if we should link to the following page, "https://blog.rackspace.com/newsroom/logos". I used the email provided on the that page, however did not link to the blog article directly.

@CITguy @araiford Thoughts? ^

Edit: Screenshot updated image

CITguy commented 6 years ago

So, it seems alright to me, but we should double check with legal. Mainly because:

  1. I'm not 100% sure if the public relations team is the appropriate contact for inquiries.
  2. I'm not sure what verbiage is needed to document the legal usage of the images.
  3. I'm not sure if we're missing something that's needed.
CITguy commented 6 years ago

With a little digging, I found https://www.rackspace.com/information/legal which has a link to https://www.rackspace.com/information/legal/copyrights_trademarks, which has a lot of useful information for public consumption. However, I'm not sure how formal we need to be given that the documentation should be just for Racker use. I'll submit an inquiry with legal to see what else we need to do.

CITguy commented 6 years ago

Also, I have a feeling that we need to change the Fanatiguy image, because it's been modified.

Edit: The fanatiguy logo has been updated on the CDN to revert it to original geometry.

technabors commented 6 years ago

@CITguy I like including the copyrights_trademarks link in this section as well.

Edit: Additionally the screenshot in the PR has been updated.

technabors commented 6 years ago

@CITguy Take a look at the rackspace gray logo on the CDN, the text has been distorted. Any chance we could get an unmodified version replacement at the same link?

http://c1ee333499ed5f44e56a-fa12562cfe810d69bedcc36a0ac289ef.r55.cf1.rackcdn.com/img/rs-logo.svg

CITguy commented 6 years ago

@technabors: that's literally the same SVG being used on rackspace.com (besides the color)

https://www.rackspace.com/profiles/www/themes/rackspace_legacy/imgs/derek/rs-logo.svg

technabors commented 6 years ago

@CITguy Ok no arguments from me then, just looked strange.