-
Notifications
You must be signed in to change notification settings - Fork 25
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
Location of Movie on Landing Page #724
Comments
I think I liked having the map full width tbh. Thoughts from others? If we do full width map, then I think we should try suggestion 2 above. |
I agree with having a full width map. It feels more consistent with the rest of the page and reduces the amount of vacant looking space.
I feel the “What is Project sidewalk?” section is too far down the page in any case.
I like option 2. of making the movie module consistent if we have the time and I would also change the position of the module to after the initial “Lets create a path for everyone” section
… On Jun 22, 2017, at 10:16 AM, Jon Froehlich ***@***.***> wrote:
I think I liked having the map full width tbh. Thoughts from others?
If we do full width map, then I think we should try suggestion 2 above.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub <#724 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADagpQHL07TLsrM2nJcwiLCKz09Sa_38ks5sGndEgaJpZM4OAUvB>.
|
What do you mean by "html container" -- is this synonymous with what I have been calling 'module' in the text above? |
Need '?'. Also, need sub-title--perhaps that says 'Click on the movie below
to learn more'
Also, Teja suggested moving this up. Thoughts on this?
…On Thu, Jun 22, 2017 at 2:26 PM, Mikey Saugstad ***@***.***> wrote:
Thoughts on the current look?
[image: what_is-separate]
<https://user-images.githubusercontent.com/6518824/27449524-b350e8cc-5756-11e7-9030-0bcdf3966f9d.png>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#724 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABi-9TxVw09GmACWz3iWqAjMIgupGyo1ks5sGrG5gaJpZM4OAUvB>
.
--
Jon Froehlich
Assistant Professor
Computer Science
University of Maryland, College Park
http://www.cs.umd.edu/~jonf/
@jonfroehlich <https://twitter.com/jonfroehlich> - Twitter
|
I think that it makes sense to have the explanation of what PS is early. My first concern would have been it looking weird with two videos butting up against each other, but with the separate title, I am no longer concerned about it. |
@misaugstad: I totally agree. I was worried about that too. I think the 'What is PS?' movie could either be the 2nd module (just after the first movie) or the 3rd module (just after the 'animated gif' style how to use PS module). Can we try both and paste what they look like here? |
Do we prefer "click on" or "check out"? And I suggest "video" > "movie" |
I agree with both of your suggestions. :) Check out and video are both better! :) |
You should use a snapshot tool that allows you to capture an entire website in a PNG like: https://addons.mozilla.org/en-US/firefox/addon/screenshot-capture-annotate/ (I believe this is also available for Chrome). In terms of preferences, I think I like this order:
And for 2., I think we can go back to what @sbower213 made for us (so almost like it is on production server but with increased opacity layer for improved text readability). |
Thanks for the tip! |
Closed via #747 |
With the choropleth on landing page, I wonder if it makes sense to move the movie somewhere? I think part of it is that it just looks a bit smashed up against the bottom of the choropleth map.
A few ways we could deal with this:
Put a whitespace (or some other colorspace) buffer between the map and movie
Similar to 1., change the movie presentation a bit so that the module is consistent with our modular sections and have a title and subtitle for the movie before the movie itself
Move the movie down to after "What people are saying" and maybe change it to "Find out more..." or something.
Thoughts?
The text was updated successfully, but these errors were encountered: