-
Notifications
You must be signed in to change notification settings - Fork 0
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
[Website] - Changes to the Landing Page & Team Page #289
Changes from 9 commits
2855e85
eecd342
a43a32c
7433ce2
a497e5f
a2a58da
d539369
cc483eb
0a6089d
5c3b634
a283b84
560121e
1db72d1
db789ef
01e8ade
6cc9241
400aef6
99a7191
5917778
7d255ec
eec1749
26baca2
fd7b7b2
4a39eeb
ddf54ea
3be8056
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this is nitpicky but can we rename this file so that it doesn't include a period in the filename itself? 4point.svg or something? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Great point -- 4.0 oftentimes uses 4pt0, so we might go with that There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. fixed it! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. perfect, nice one |
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Similarly, if we are using hyphens between words (like with andy-circle.png) we should probably be consistent throughout the folder. So this would be coming schmidt-futures.svg There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. just renamed it with the rest of the logos! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. sounds good |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you both need to make these changes to get the environment running? It's probably not a big deal but can you provide some context to explain why each of the versions are changing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We actually don't know what happened but currently we both have the node-sass: 8.0 version on our local folder.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmmm, ok sounds good. let's go with these versions then.