studio icon indicating copy to clipboard operation
studio copied to clipboard

Update readme structure, image header, and links

Open ocin-rye opened this issue 3 years ago • 12 comments

User-Facing Changes

Update readme structure, image header, and links

Description

Changes made:

  • New header image, title, links, and badges
  • The 'contribute' section has been moved above the 'self-host' section (Esther mentions that contribution is a higher priority and self-host)
  • Two sets of vertical links sets have been changed to horizontal links to reduce page length

Thing(s) to consider:

  • I've kept the existing repo description with the links as is, but they may be redundant with the newly added links: "Foxglove Studio is an integrated visualization and diagnosis tool for robotics, available in your browser or for download as a desktop app on Linux, Windows, and macOS."

Addresses #4615 - Style uplift on Studio Readme

ocin-rye avatar Oct 14 '22 00:10 ocin-rye

I guess there’s probably not much that can be done to make the new header image look less small on mobile?

97D6B3C6-2DF1-4500-AF26-8B2996C39D25

jtbandes avatar Oct 14 '22 16:10 jtbandes

Actually I wonder if we could use this media attribute with a min-width/max-width: https://docs.github.com/en/get-started/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax#specifying-the-theme-an-image-is-shown-to

jtbandes avatar Oct 14 '22 16:10 jtbandes

Wonder if it should be either “Accelerate Your Robotics Development” or “Accelerate your robotics development”? On the website homepage we use “Accelerate your robotics development” (purple).

jtbandes avatar Oct 14 '22 16:10 jtbandes

@jtbandes

  • I'll take look and see if the media attribute is a possible solution for the image responsiveness.
  • Good catch, I'm in favor of correcting the title casing to read “Accelerate Your Robotics Development”, but using sentence case makes sense too for alignment with the homepage. @2metres @esthersweon Any preferences or existing branding guidelines for this?

ocin-rye avatar Oct 14 '22 17:10 ocin-rye

I think we have leaned towards sentence case for strings in our products, but not aware of any existing guidelines around web content/docs

jtbandes avatar Oct 14 '22 18:10 jtbandes

Sounds good, I'll update to sentence case for now

readme-header

ocin-rye avatar Oct 14 '22 18:10 ocin-rye

It would be awesome to add an equivalent image to the https://github.com/foxglove/mcap readme too, especially since we're about to present mcap at a conference so it'll be getting a lot more traffic 🔜!

jtbandes avatar Oct 14 '22 20:10 jtbandes

Uploading the mobile image to reference URL in the readme file

readme-header-mobile

ocin-rye avatar Oct 14 '22 21:10 ocin-rye

Looks like there's some weird height mismatch issue at the top of that image

image

jtbandes avatar Oct 14 '22 22:10 jtbandes

I didn't see that in dark mode. This image should resolve it. readme-header-mobile

ocin-rye avatar Oct 14 '22 22:10 ocin-rye

I think it's a little strange that we only include "Web app" in the nav links, but then include links to both apps in the following sentence:

Screen Shot 2022-10-16 at 12 35 21 PM

Could we just have "Download" in the nav link to take you to a page for both apps, or have both links in the nav? Or maybe exclude both, since the first sentence takes care of calling out both?

That first sentence is also worded a little awkwardly (I'd recommend "available in your browser or as a desktop app").

esthersweon avatar Oct 16 '22 19:10 esthersweon

I think it's a little strange that we only include "Web app" in the nav links, but then include links to both apps in the following sentence:

Screen Shot 2022-10-16 at 12 35 21 PM

Could we just have "Download" in the nav link to take you to a page for both apps, or have both links in the nav? Or maybe exclude both, since the first sentence takes care of calling out both?

That first sentence is also worded a little awkwardly (I'd recommend "available in your browser or as a desktop app").

@esthersweon 👍 I've added this copy/link update

ocin-rye avatar Oct 17 '22 14:10 ocin-rye

Great stuff!

defunctzombie avatar Oct 18 '22 20:10 defunctzombie