Skip to content
This repository has been archived by the owner on Nov 24, 2020. It is now read-only.

Add Mike's photo and bio, etc. #6

Merged
merged 5 commits into from
Jun 10, 2015
Merged

Add Mike's photo and bio, etc. #6

merged 5 commits into from
Jun 10, 2015

Conversation

rgulle4
Copy link
Contributor

@rgulle4 rgulle4 commented Jun 7, 2015

Add Mike's photo and bio, and other changes to the about page.

Also changed Jonathan's headshot at his request. Also made the indent style consistent throughout the page (I assumed that tab chars are preferred).

Closes #4. And hopefully gets #5 right.

- Add Mike's headshot
- Add Mike's bio and headshot to the about page.
- Change all indents to tab chars, for consistency.
- Previous commit had both tab indents and 2-space indents.
- This and other pages strongly prefer tab indents (not entirely consistent though).

FTR, I like space char indents myself. In either case, it might be a good idea to do a site-wide style refactor.
@lleger
Copy link
Member

lleger commented Jun 7, 2015

Mike is our Creative Director. Did he give you the bio or did you write it?

We use soft tabs. 2 spaces. So that should be changed back.

@jmhooper
Copy link

jmhooper commented Jun 9, 2015

I grabbed the bio from a BN marketing document. I added the first sentence.

@rgulle4
Copy link
Contributor Author

rgulle4 commented Jun 9, 2015

We use soft tabs. 2 spaces. So that should be changed back.

Ok. (I also opened #8 to do this for the whole site).

<h1>&ldquo;The Guys&rdquo;</h1>
<section id="logan">
<h2>Logan Leger</h2>
<%= image_tag("logan_headshot.png", :width => "125px") %>
Copy link
Member

Choose a reason for hiding this comment

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

You can use the new hash syntax in Ruby 1.9:

image_tag("logan_headshot.png", width: "125px")

@lleger
Copy link
Member

lleger commented Jun 10, 2015

Change Mike's bio to the following:

Mike is the Creative Director, conceptualizing, wireframing, and designing all of our web and mobile projects. He has a varied background, with experience in everything from brand strategy and execution to interactive design, digital product development, and visual content design. Mike hails from Cleveland, and in his spare time likes to wallow in the misery that is Cleveland sports.

Also, @jmhooper graduated, so let's update his bio. Change:

He studies computer science and math at Louisiana State University.

to:

He graduated from Louisiana State University with a degree in computer science, with a focus in software engineering.

And change:

When he isn’t behind his laptop working on a project or schoolwork, he enjoys spending time outdoors hiking and backpacking.

to:

When he isn’t behind his laptop working on a project, he enjoys spending time outdoors hiking and backpacking.

Make those changes and then I think this is good to go.

@rgulle4
Copy link
Contributor Author

rgulle4 commented Jun 10, 2015

Done.

rgulle4 added a commit that referenced this pull request Jun 10, 2015
@rgulle4 rgulle4 merged commit a203d33 into master Jun 10, 2015
@rgulle4
Copy link
Contributor Author

rgulle4 commented Jun 10, 2015

Accidentally an extra </section> tag. Just fixed it in master. f4c2c56

@lleger
Copy link
Member

lleger commented Jun 11, 2015

Usually, before we merge, we do a squash, so that we clean up the git history a little. When you do some fix up commits, like you have here, it just adds noise, so we streamline those into one commit so it makes more sense.

But it looks good! I just deployed it so you can see it live. Good job. 🎉

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.

None yet

3 participants