Skip to content
This repository has been archived by the owner on Dec 6, 2024. It is now read-only.

Update pictures, add Samar, change format. #121

Merged
merged 4 commits into from
Jan 6, 2017

Conversation

kelnera
Copy link
Contributor

@kelnera kelnera commented Jan 3, 2017

@osuosl/web-reviewers All the current pictures have now been added to the OSL website as well, including a picture for Samar.

Bigger change, I've changed the line ':scale: 50%' to ':width: 110 px', like it is on the CASS website. 50% only works if we have consistency in the original pictures, which we still don't so it looks uneven. Please let me know if you have any objections or concerns. Also please double check that the new pictures look right and are showing up correctly. Below are the list of changed pictures.

Justin Dugger
Samar Hedaoo
Amanda Kelner
Annie Lei
Spencer Moss
Tristan Patch
Hannah Solorzano
Andy Tolvstad
Levi Willmeth

@kelnera
Copy link
Contributor Author

kelnera commented Jan 3, 2017

I have also included removing the TriSano paragraph from 'services_development' and DNS paragraph from 'hosting_details' as requested in #51.

Copy link
Member

@ramereth ramereth left a comment

Choose a reason for hiding this comment

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

The content looks good but you've updated the submodule reference to an older version. Can you please remove that update?

@kelnera
Copy link
Contributor Author

kelnera commented Jan 3, 2017

@ramereth I'm not sure what you're looking at. The only reference to a submodule says there's nothing to compare.

@ramereth
Copy link
Member

ramereth commented Jan 4, 2017

@kelnera it might be hard to see from the github gui. Do you mind if I fix it for you?

@kelnera
Copy link
Contributor Author

kelnera commented Jan 4, 2017

@ramereth Nope, that's fine by me!

@kelnera kelnera merged commit 1a14c1b into master Jan 6, 2017
@ramereth ramereth deleted the kelnera/Staff-Pictures-Jan-2017 branch August 11, 2017 23:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants