Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add favicon to style guide #377

Merged
merged 2 commits into from
Apr 4, 2019
Merged

add favicon to style guide #377

merged 2 commits into from
Apr 4, 2019

Conversation

josephgknox
Copy link

READY FOR REVIEW

Summary

  • Adds Favicon to our style guide site (decanter.stanford.edu), which currently does not have one

Needed By (Date)

  • N/A

Urgency

  • N/A

Steps to Test

  1. Pull this branch
  2. Compile styleguide: npm run styleguide
  3. Verify that your local site now has a Favicon (Stanford's)

Affected Projects or Products

  • Decanter

Associated Issues and/or People

  • N/A

See Also

@josephgknox
Copy link
Author

CC: @kerri-augenstein (FYI that this is being worked on).

@kerri-augenstein
Copy link

Testing @josephgknox.

Copy link
Collaborator

@JBCSU JBCSU left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GTG. Thanks, @josephgknox!

@sherakama sherakama merged commit d6910ca into master Apr 4, 2019
@sherakama sherakama deleted the favicon branch April 4, 2019 16:36
yvonnetangsu added a commit that referenced this pull request Jun 11, 2019
* master: (44 commits)
  Set up color maps for Decanter and refactor main nav colors to refer to color maps (#420)
  Move PR template (#416)
  Fix skiplink target so it doesn't take up space and cause overflow (#415)
  Release 5.0.1 (#417)
  Fix CSS grid gaps not displaying on Edge issue and other minor grid related issues (#413)
  More link fixup (#388)
  forgot to rebuild .js after final lint fixes (#387)
  add openNav, closeNav, openSubnav, closeSubnav events (#379)
  Fix main nav color variants and work with new linting rules (#380)
  add favicon to style guide (#377)
  Adjust linters. (#370)
  update readme.md to match Scott's intro on homepage.md (#371)
  Update README.md (#369)
  move the images we need for the styleguide's homepage into kss-assets (#368)
  Update README.md (#367)
  Update UPGRADE.md (#363)
  package.locl.json file is back (#366)
  Added more to changelog (#364)
  Update homepage.md (#362)
  90 basic webpack (#334)
  ...

# Conflicts:
#	core/scss/components/index.scss
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants