nextcloud / nextcloud.com

🌏 Our website
https://nextcloud.com
Other
208 stars 127 forks source link

update screenshots for 15 #1020

Open jospoortvliet opened 5 years ago

jospoortvliet commented 5 years ago

We need to update screenshots for 15. Ideally, all get taken again, but that is hard in some cases, requiring a call etc. Luckily we have many screenshots taken by @MariusBluem - so many features are already covered, just need page updates!

What needs to happen is:

  1. make sure there's a <div class="row"> with in it a <div class="col-md-6">
  2. Add to the col-md-6 the following classes: image--feature (might be there already) and new-img
  3. update image URL! make sure image has the class featureimg
  4. Test! If it doesn't look good, double-check there's a row and it is col-md-6, others are not (yet) supported.

Please help, @nextcloud/website

Pages to do:

the product pages

Our feature pages

Other pages

Our various industries

jancborchardt commented 5 years ago

@MariusBluem @jospoortvliet @nextcloud/website for any new screenshots we need to make sure:

This especially makes it easier to view them on mobile (narrow screens), but also for everyone generally.

Compare this (currently on https://nextcloud.com/secure/):

And this screenshot I just made on full HD with 180% zoom in Firefox (yes, Ubuntu doesn’t handle High-DPI properly, but the point is that everything is visible and there’s no unnecessary whitespace on the right): nextcloud 180

jospoortvliet commented 5 years ago

@jancborchardt the new screenshots don't have the blue anymore, I add the 'browser chrome' and the background with css. On mobile screens, the blue doesn't show at all. We can change the look later, I don't want to break the design because it does look nice.

Zoom in the screenshots is fine - as much as wanted, it doesn't matter much ;-)

As long as the screenshot is 16:9.

MariusBluem commented 5 years ago

I will push updated screenshots tonight, with 16:9, zommed in and all the other requirements. 📸📸

MariusBluem commented 5 years ago

Please review if this would be OK: https://cloud.nextcloud.com/s/w2PzwkqKzsi4F2F

More than 170% seems to be a bit messy (layout-wise)

jospoortvliet commented 5 years ago

@MariusBluem yes, that all looks good! Run them through https://tinypng.com & put them in a PR with the html changes (if you can, if not, I can help).

Great work!

jancborchardt commented 5 years ago

These look great @MariusBluem, thank you so much! Finally nicely sized screenshots with visible content! :wink: