Navigation Menu

Skip to content
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

New homepage #555

Merged
merged 13 commits into from May 15, 2017
Merged

New homepage #555

merged 13 commits into from May 15, 2017

Conversation

sethladd
Copy link
Contributor

@sethladd sethladd commented May 12, 2017

@googlebot googlebot added the cla: yes Contributor has signed the Contributor License Agreement label May 12, 2017
@FlutterBot
Copy link

Staging URL Generated At https://flutter-io-deploy-two.firebaseapp.com . Please allow Travis Build to finish to view the URL.

@Hixie
Copy link
Contributor

Hixie commented May 12, 2017

feedback from someone looking at this on mobile: there's no left margin, the text starts at pixel 0

@sethladd
Copy link
Contributor Author

Thanks!

@Hixie
Copy link
Contributor

Hixie commented May 12, 2017

If I make my page narrow on desktop, the text goes off the left edge entirely.

@jhirth
Copy link

jhirth commented May 12, 2017

There is no prominent link to the FAQ (/faq/). Was this a deliberate omission?

@sethladd
Copy link
Contributor Author

Good catch on the FAQ, thanks!

@sethladd
Copy link
Contributor Author

Added FAQ.

Known issue: the footer behaves weirdly sometimes. Will address in future PR.

Identified the source of the non-responsiveness. Can fix in this PR.

Can fix the top toolbar not responding soon enough in this PR.

index.md Outdated
<div class="homepage__key_point_title">Used by Google</div>

<p>
Flutter is used by Google, works with Firebase
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe it's worth saying Flutter is used by Google and others? I'm a little worried that some people wonder if it's used by Google only.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added "and others"

</div>

<div class="homepage__key_point">
<div class="homepage__key_point_title">Used by Google</div>
Copy link
Contributor

Choose a reason for hiding this comment

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

Will "Used in production apps" better capture the content (e.g., Firebase and other SDKs) below this heading?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added "in production"

index.md Outdated
</section>

<section class="homepage__hot_reload">
<h1>Fast development</h1>
Copy link
Contributor

Choose a reason for hiding this comment

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

Adding the lightening bolt icon next to "Fast development" is likely to help users discover the feature, once they are in the IDE.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can you link me to the icon that we have the rights to use? Happy to add it here. Bonus if it's an SVG that I can embed :)

Copy link
Contributor

Choose a reason for hiding this comment

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

@devoncarew Do you have the source image of the Hot Reload icon?

Copy link
Member

Choose a reason for hiding this comment

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

we have a 32x32 png (https://github.com/flutter/flutter-intellij/blob/master/resources/icons/hot-reload%402x.png)

If you want a larger image, there is one that's similar but not identical which I could dig up.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

thanks!

<p>
Delight your users with Flutter's built-in
beautiful Material Design and
Cupertino (iOS-flavor) widgets, rich motion APIs,
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we have GIF or embedded video to show "motion" and smooth scrolling? At the minimum, a link to a video demo on YouTube would help support our claims.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Totally. I want to try to use HTML5 video for this.

index.md Outdated
</p>

{% prettify dart %}
class MyMessage extends StatelessWidget {
Copy link
Contributor

Choose a reason for hiding this comment

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

This snippet is short and sweet, but it doesn't show the reactiveness of the framework. How about use this snippet: https://flutter.io/widgets-intro/#changing-widgets-in-response-to-input?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Let's give it a try.

{% endprettify %}

<p>
Browse the <a href="/widgets/">widget catalog</a>
Copy link
Contributor

Choose a reason for hiding this comment

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

It might be more useful to have the link to the Widget Catalog in the "Expressive, beautiful UIs" section.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Can do.

typography, icons, and more.
<p>
Learn more about what makes Flutter special in the
<a href="/technical-overview/">technical overview</a>.
Copy link
Contributor

Choose a reason for hiding this comment

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

This page currently doesn't show up in the side navigation.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch. Added.

## Why use Flutter?
<div class="homepage__button_row">
<a href="/getting-started/" class="get-started-button">Get Started</a>
{% include note.html content="_Flutter is an alpha, open-source project_." %}
Copy link
Contributor

@InMatrix InMatrix May 12, 2017

Choose a reason for hiding this comment

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

Add something about how to give us feedback here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We used to have that, but now there's a Support link on every page on the website.

@InMatrix
Copy link
Contributor

@sethladd Thank you for putting this new homepage together. It looks very nice overall. Please see my inline comments for some potential tweaks.

@christian-lefty
Copy link

I would suggest that most of the stuff in the Unified app development section be links to relevant doc

@sethladd
Copy link
Contributor Author

I would suggest that most of the stuff in the Unified app development section be links to relevant doc

Thanks! One of the things that this exercise is pointing out is that we need more docs for some things, and to bring some docs over to flutter. :)

Copy link
Member

@mit-mit mit-mit left a comment

Choose a reason for hiding this comment

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

Just some smaller comments. Overall a great improvement!

I wonder if the 'cards' would look more consistent with the overall design if they used the same styling as the smaller ones here: https://flutter-io-deploy-two.firebaseapp.com/widgets/


We're open source and would love to hear from you!

We encourage you to reach out and
Copy link
Member

Choose a reason for hiding this comment

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

More short lines

Copy link
Contributor Author

Choose a reason for hiding this comment

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

short lines are fantastic for tech writer reviews. I was trained to do this :)

@@ -1,213 +1,297 @@
---
title: Welcome to Flutter
sidebar: home_sidebar
title: Easy and fast SDK for beautiful apps
Copy link
Member

Choose a reason for hiding this comment

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

s/apps/mobile apps/?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

maybe? do people think apps != mobile, these days?

index.md Outdated

<section class="homepage__key_points">
<h1 class="homepage__title">
Flutter makes it easy
Copy link
Member

Choose a reason for hiding this comment

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

Why are there so many super short lines? Why not just wrap at 80?

// build methods fast, so that you can just rebuild anything that
// needs updating rather than having to individually change
// instances of widgets.
return new Row(
Copy link
Member

Choose a reason for hiding this comment

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

This code doesn't actually compile. It complains that a RaisedButton needs to be inside a Material parent.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch. I think I took this from another page on our site.

Copy link
Member

Choose a reason for hiding this comment

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

Looks like this still isn't fixed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not clear this needs to have a Material. Can you have a Material inside another Material? This widget is intended to be inside another build() of something that has material.

But, we'll merge, and chat about a better sample.

</p>

{% prettify dart %}
// From https://github.com/flutter/flutter/tree/master/examples/platform_channel
Copy link
Member

Choose a reason for hiding this comment

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

Maybe move the 'From' link outside the code section, and make it a link to the sample.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think I tried that, but I'm happy to try again.

existing Java/ObjC/Swift investment.
</p>

<div class="feature-lists">
Copy link
Member

Choose a reason for hiding this comment

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

These lists look a little odd. Maybe they should be links?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

One insight from this exercise is that we have a ton of awesome capabilities in our system, but we don't always have docs that link to the specific feature or capability. Agreed that many of these items can and should be links. I'll try to find links for the subjects that do have a natural documentation for them.

@sethladd
Copy link
Contributor Author

@mit-mit not sure I'm going to keep the "cards". If we do, agreed we should make them consistent.

@sethladd
Copy link
Contributor Author

@InMatrix @eseidelGoogle @mit-mit I'm at the diminishing returns stage of this PR. Can you PTAL ?

After this lands, I want to fix the code samples on small screens. They should scroll horizontally. And, add more links for the feature list.

@eseidelGoogle
Copy link
Contributor

Do I need to wait for the stage bot to do it's thing?

@sethladd
Copy link
Contributor Author

@eseidelGoogle
Copy link
Contributor

eseidelGoogle commented May 15, 2017

The code example in the "Modern, reactive framework" doesn't feel very compelling.
The "Unified app development" section has a wall of buzzwords, but I'm not 100% sure what it's telling me (SEO?)

But honestly this is much better at selling Flutter (rather than explaining/onboarding as we'd previously chosen to do), so I would just press the button and ship it. lgtm. We will iterate from here.

@sethladd
Copy link
Contributor Author

doesn't feel very compelling

Any thoughts on would would make it more compelling? We can write something custom for the homepage.

cc @InMatrix

wall of buzzwords

This is intended to do a few things:

  1. Convince new users that we're "full stack". We have what you need.
  2. SEO
  3. Provide a basis for further docs and links

@mit-mit
Copy link
Member

mit-mit commented May 15, 2017

My only remaining concern is that the code snippet shown under Modern, reactive framework still isn't valid code, but I'd be to see a bug opened for that and then handle that post merge.

LGTM

@sethladd
Copy link
Contributor Author

Thanks for taking a look!

@sethladd sethladd merged commit 775bad8 into master May 15, 2017
@sethladd sethladd deleted the new-homepage branch May 15, 2017 18:14
@InMatrix
Copy link
Contributor

Sorry about the delayed response. Yes, LGTM. As I mentioned previously, I'd love to see animations to show our performance and motion capability in the "Expressive, beautiful UIs" card.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Contributor has signed the Contributor License Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants