Skip to content

embedded: add text, icons, testimonials and friends section #62

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

Merged
merged 6 commits into from
Jul 25, 2018

Conversation

japaric
Copy link
Member

@japaric japaric commented Jul 23, 2018

The layout is the one we discussed during Rust All Hands plus a friends section at the end.

This is what it looks like so far:

screen shot 2018-07-23 at 18 18 32-fullpage

cc @jamesmunns

<div class="four columns">
<div class="domain-icon">
<!-- TODO replace this icon; this is just a sample -->
<img src="https://image.flaticon.com/icons/svg/63/63794.svg" width="128" alt="A microscope"/>
Copy link
Member Author

Choose a reason for hiding this comment

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

We discussed what icons would fit with the Rust attributes and came up with these ideas (e.g. microcospe) after some brainstorming; suggestions are welcome.

The icon I included here is the first thing I found on the internet; we likely will want to pick something that has an appropriate license, or have the icon made.

Copy link
Member

Choose a reason for hiding this comment

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

these looks pretty great, i have already attributed the page you used in the footer, https://www.flaticon.com/, so i think we're actually all set?

Copy link
Member Author

@japaric japaric Jul 24, 2018

Choose a reason for hiding this comment

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

Alright, then we can use these icons.

<h3><a href="#">Company #3</a></h3>
"Cargo is pretty much worth it to switch by itself."

<!-- Friends section -->
Copy link
Member Author

Choose a reason for hiding this comment

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

We probably want some text or blank space between the testimonials and the friends section. It looks a bit cluttered right now, IMO.

Copy link
Member

Choose a reason for hiding this comment

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

if you throw this in a <section> the spacing should be fixed :)

<h2>Embedded Rust in production</h2>
</header>

<!-- Testimonials section -->
Copy link
Member Author

Choose a reason for hiding this comment

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

I have included only two testimonials but there are two companies that might be able to provide two more testimonials. We discussed this section and agreed that having a carousel of testimonials like this one would save vertical space in case we end up with several testimonials.

That discussion was before I saw the testimonial style that's being used in the front page (and which I copy pasted here).

Copy link
Member

Choose a reason for hiding this comment

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

yeah i just did something so that it looked vaguely ok for the moment :) i'm happy to add a carousel if you think you'd prefer!

Copy link
Member Author

Choose a reason for hiding this comment

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

I'd love to see how a carousel would fit with the rest of the page.

Copy link
Member

@ashleygwilliams ashleygwilliams left a comment

Choose a reason for hiding this comment

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

hey @japaric this is a great start- i think you are really close to the Stage 1, just a few markup changes :) after that i can merge and help fix up some of the CSS and ya'll should be ready for stage 2 with @aturon - thanks so much!

@@ -1,116 +1,245 @@
{{#*inline "page"}}

Copy link
Member

Choose a reason for hiding this comment

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

can you add a header that says "Embedded Devices" here? similar to this: https://github.com/rust-lang/wubwub/blob/master/templates/policies/media-guide.hbs#L3-L7

<div class="four columns">
<div class="domain-icon">
<!-- TODO replace this icon; this is just a sample -->
<img src="https://image.flaticon.com/icons/svg/63/63794.svg" width="128" alt="A microscope"/>
Copy link
Member

Choose a reason for hiding this comment

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

let's not hard code the width :) i can handle that with some css :)

<!-- TODO replace this icon; this is just a sample -->
<img src="https://image.flaticon.com/icons/svg/908/908319.svg" width="128" alt="A RAM stick"/>

<h5>Flexible memory management</h5>
Copy link
Member

Choose a reason for hiding this comment

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

could you make these h3s? if you want them smaller we can handle that in CSS :)

<img src="https://image.flaticon.com/icons/svg/1/1615.svg" width="128" alt="DIP-6 package"/>

<a href="https://japaric.github.io/discovery">
<h5>The Discovery book</h5>
Copy link
Member

Choose a reason for hiding this comment

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

h3s again :)

<h3><a href="#">Company #3</a></h3>
"Cargo is pretty much worth it to switch by itself."

<!-- Friends section -->
Copy link
Member

Choose a reason for hiding this comment

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

if you throw this in a <section> the spacing should be fixed :)

@japaric japaric changed the title [WIP] embedded: add text, icons, testimonials and friends section embedded: add text, icons, testimonials and friends section Jul 24, 2018
@japaric
Copy link
Member Author

japaric commented Jul 24, 2018

@ashleygwilliams I think I addressed all your comments. Let me know if I missed something!

@ashleygwilliams
Copy link
Member

cool! i'm going to merge and do a little bit of CSS but this is definitely a pass for stage 1! thanks so much

@ashleygwilliams ashleygwilliams merged commit c123e9f into master Jul 25, 2018
@ashleygwilliams ashleygwilliams deleted the embedded branch July 25, 2018 14:16
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.

2 participants