Starter showcase - detail page wireframes

[shannonbux]

Sub-tasks:

• Move “visit site” and social sharing button above screenshot (Flo can provide design) Issue Revisit detail page header layout - Starter Showcase #6710 [www/showcase] Revisit detail page header layout #6098
• Add prev/next links Add prev/next links to detail page - starter showcase #6711 (edit: we're not going to do this right now because we're not sure people will browse through starters in the same way they do the site showcase)
• Shannon to decide → Pull in the README of the package for the “description” field or ask authors to write descriptions? Ok decided authors to write descriptions. Decide how to fill "description" field - starter showcase #6712
• Hover state of deploy to netlify looks funky! Follow pattern we use for other buttons Hover state of "deploy to netlify" button needs to match other buttons - starter showcase #6713
• external dependencies - icon at end instead of beginning, selected state to include whole word update external dependencies on detail page - starter showcase #6716
• drop shadow around screenshot to match site showcase detail page Drop shadow on detail page of starter showcase #6766

[swyxio]

fwiw i would not recommend pulling in the README as there are no limitations on how they are formatted and its really hard to make that look presentable. for example what if the readme includes a screenshot of the demo? then our page has two screenshots

[swyxio]

fixing

• external dependencies - icon at end instead of beginning, selected state to include whole word update external dependencies on detail page - starter showcase #6716
• Hover state of deploy to netlify looks funky! Follow pattern we use for other buttons Hover state of "deploy to netlify" button needs to match other buttons - starter showcase #6713

waiting for more input

• Move “visit site” and social sharing button above screenshot (Flo can provide design) Issue Revisit detail page header layout - Starter Showcase #6710 [www/showcase] Revisit detail page header layout #6098
• Add prev/next links Add prev/next links to detail page - starter showcase #6711

feedback

drop shadow around screenshot to match site showcase detail page #6766

there is no drop shadow in the direct page view (example); it only shows up in the modal view. last i recall we decided against having the modal view for the starter showcase.

[shannonbux]

Ok instead of the READme we'll just have a field that can be filled out in the .yml file.

@fk can you answer the question above about the modal views:

it only shows up in the modal view. last i recall we decided against having the modal view for the starter showcase.

[swyxio]

re: yml vs readme

yea so fwiw i'm working with markdown instead of yml files for this exact purpose. when they submit they can choose to fill out the body of the markdown with whatever they want and we can choose to display it in future. its an intentional point of departure from the site showcase because i believe this is better long term. i struggle with it because of this difference but kyle knows i'm doing this and has never told me otherwise so i stuck with what i had for now (since i already did all the manual data entry).

[shannonbux]

oh ok! I'll make sure the instructions make this clear :)

…

On Fri, Jul 27, 2018 at 3:07 PM, shawn wang ***@***.***> wrote: re: yml vs readme yea so fwiw i'm working with markdown instead of yml files for this exact purpose <https://github.com/sw-yx/gatsby/tree/starterShowcase2/www/src/data/StarterShowcase/startersData>. when they submit they can choose to fill out the body of the markdown with whatever they want and we can choose to display it in future. its an intentional point of departure from the site showcase because i believe this is better long term. i struggle with it because of this difference but kyle knows i'm doing this and has never told me otherwise so i stuck with what i had for now (since i already did all the manual data entry). — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#6709 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/Ae9o2vPcQx9np0mbHdj6ItTBRQzua567ks5uK4D5gaJpZM4Vd77q> .

[fk]

@sw-yx

drop shadow around screenshot to match site showcase detail page #6766

there is no drop shadow in the direct page view (example); it only shows up in the modal view. last i recall we decided against having the modal view for the starter showcase.

There is a subtle box-shadow for the screenshot in the permalink/direct page view: https://next.gatsbyjs.org/showcase/reactjs.org/. It's there to separate screenshots that contain a lot of white from the white page background.

[swyxio]

ok. i'm happy to push the last mile on this if you are willing to work with me on it, but im not sure what to do about it now that it has been moved to backlog.

[shannonbux]

Hi @fk! Could you just add any new mockups or notes on what you're planning to do here? Thanks!

[amberleyromo]

@shannonbux --

@fk dropped mockup details over on this issue: #6710 (comment)

Not sure if you want to consolidate issues here, perhaps

[shannonbux]

Closing in favor of #6710