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

Collage for website and README #1596

Merged
merged 12 commits into from Jun 28, 2017
Merged

Collage for website and README #1596

merged 12 commits into from Jun 28, 2017

Conversation

jlstevens
Copy link
Contributor

Adding a collage of nice examples to the website and README.

@jbednar
Copy link
Member

jbednar commented Jun 22, 2017

Looks good!

@jlstevens
Copy link
Contributor Author

Here is the preview when done from master (the preview from the PR doesn't spread across rows for some reason):

collage

@jlstevens
Copy link
Contributor Author

jlstevens commented Jun 22, 2017

This would be a good time to upload the logo so we can put it next to the HoloViews header (I suggest capitalizing it). Would also be good to update the entire README...

@jlstevens
Copy link
Contributor Author

jlstevens commented Jun 22, 2017

@philippjfr @jbednar Any idea how to get the logo on the same line as the heading in rst? I get this which is ugly:

image

using:

.. raw:: html

     <img src="http://assets.holoviews.org/logo/holoviews_color_icon_500x500.png" alt="HoloViews logo" height="70px" />

HoloViews
=========

It is possible (at least with Markdown). See nteract's README for instance.

@jbednar
Copy link
Member

jbednar commented Jun 22, 2017

No idea. Should the gallery include at least one plot showing a set of numerical sliders on a HoloMap (or DynamicMap)? That's such a big selling point of HV that it's odd it's not included here.

@jlstevens
Copy link
Contributor Author

Should the gallery include at least one plot showing a set of numerical sliders on a HoloMap (or DynamicMap)

Do you think it has to be animated? Or would a static one with a slider get the point across?

@jbednar
Copy link
Member

jbednar commented Jun 22, 2017

Either is probably fine.

@jlstevens
Copy link
Contributor Author

In that case I suggest removing the linked stream example in the middle (bottom row) and replacing it with something that has a slider (and maybe some sort of linked stream).

@jlstevens
Copy link
Contributor Author

Updated to show one with a slider:

collage

@jlstevens
Copy link
Contributor Author

@philippjfr @jbednar Are we happy with this before I convert the README to markdown?

@philippjfr
Copy link
Member

philippjfr commented Jun 22, 2017

@philippjfr @jbednar Are we happy with this before I convert the README to markdown?

Can we make the bottom row a tiny bit larger, basically scale them all up until their overall width right-aligns with the legend of the top row. Could even add a little bit of whitespace to the top row to give you more scope to scale up the bottom row.

@jlstevens
Copy link
Contributor Author

jlstevens commented Jun 26, 2017

All the examples in this collage are now examples in #1537. I would like to update some of the thumbnails which have changed and add links to where these examples will be before this PR is merged.

Edit: Not quite true - the cell's example is from the getting started guide. Probably not worth turning into an example? What do you think @philippjfr ?

@jlstevens
Copy link
Contributor Author

This PR will address #1596 once merged.

@jlstevens
Copy link
Contributor Author

Every image in the collage now links to the page where that example can be found.

Note that the http://build.holoviews.org urls will need to be replaced with http://holoviews.org once the website is live. As we need to update the homepage with new content in a separate PR, this doesn't need to be changed now.

Here is what it looks like now:

image

@philippjfr Do you agree we should merge now then finish up the homepage?

@philippjfr
Copy link
Member

Happy to merge this now, will still have to optimize file sizes for these but lets get it in place first.

@philippjfr philippjfr merged commit ae355d5 into master Jun 28, 2017
@jlstevens
Copy link
Contributor Author

The gifsicle utility seems to work well for gifs. Not sure what to do about pngs.

We should optimize these before the final homepage update.

@philippjfr
Copy link
Member

I noticed the gifs were still fairly large even after you resized them, going to try a few other things.

@jlstevens
Copy link
Contributor Author

I resized the gifs so their width was 500 pixels i.e they are still probably larger than they need to be.

@philippjfr
Copy link
Member

Generally as long as they don't contain images the gifs compress very well so I'll try some compression tomorrow.

@philippjfr philippjfr deleted the collage branch July 31, 2017 22:57
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.

None yet

3 participants