Skip to content
New issue

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

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

Already on GitHub? # to your account

removed FA from project; updagraded to latest normalize #101

Merged
merged 2 commits into from
Apr 27, 2018

Conversation

josephgknox
Copy link

@josephgknox josephgknox commented Apr 20, 2018

READY FOR REVIEW

Summary

  • Removed FontAwesome from project entirely
  • Upgraded to latest version of normalize

Needed By (Date)

  • N/A

Urgency

  • N/A

Steps to Test

  1. Pull this branch
  2. Ensure FA is gone
  3. Ensure new normalize is added (8.0.0)

Affected Projects or Products

  • Decanter
  • Stanford Basic

Associated Issues and/or People

See Also

@josephgknox
Copy link
Author

Cc: @pookmish

@sherakama
Copy link
Member

This looks good to me.

@JBCSU & @AlishaOber
This PR removes Font Awesome, and subsequently, 2300 lines of css from the rendered output as FA is not being utilized in any of the components at the moment.

This also updates to the latest version of normalize.css.

Please have a review and let me know if there are any issues with these changes.

Copy link
Member

@sherakama sherakama left a comment

Choose a reason for hiding this comment

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

This looks good to me.
Wait for feedback from JBC and Alisha before merging.

@pookmish
Copy link
Contributor

@josephgknox looks like some conflicts in this now.

@josephgknox
Copy link
Author

@pookmish yeah, thanks for keeping your eyes on this. We had merged in another branch off of master that still have the legacy stuff in there (since this one has not been merged). I've resolved the conflicts.

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.

Looks great!

@sherakama sherakama merged commit e7783f3 into master Apr 27, 2018
@sherakama sherakama deleted the bye-fa-hello-new-normalize branch April 27, 2018 20:11
@sherakama
Copy link
Member

Wonderful. Thank you.

josephgknox pushed a commit that referenced this pull request Apr 27, 2018
* Quote tweaks (#105)

* Created quote element within elements directory based on design Kerri made for the Engineering project

* tweak Quote template to make it easier to include directly into themes

* don't hardcode path to quote's image

* update neat-omega, grunt, grunt-sass, jquery, and associated dependencies (#106)

* #107 specify node version (#108)

* removed FA from project; updagraded to latest normalize (#101)
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants