neo4j-documentation / docs-ui

Mozilla Public License 2.0
4 stars 23 forks source link

Add GraphGists UI #51

Closed ggrossetie closed 3 years ago

ggrossetie commented 3 years ago
ggrossetie commented 3 years ago

Thanks @adam-cowley for your review/updated design! I will add a GraphGist category page in the ui-model.yml to make sure that it looks good (using the preview).

ggrossetie commented 3 years ago

@adam-cowley I think we are good to go? We can still adjust the design if pages don't look as good with the actual data from the portal.graphgist.org API.

Index page

localhost_5252_graph-gist-index html(Laptop with HiDPI screen)

Category page

localhost_5252_graph-gist-category-pop-culture html(Laptop with HiDPI screen)

GraphGist page

localhost_5252_graph-gist-star-wars-social-network html(Laptop with HiDPI screen)

adam-cowley commented 3 years ago

Those cards could probably do with having a white background but apart from that I'm happy to go live with these

ggrossetie commented 3 years ago

@adam-cowley I found a few issues with the runnable code block component but I'm not entirely sure how to fix them. Maybe we could merge this pull and deploy a first version of the GraphGist site in a staging environment (Netlify?) in order to gather feedback and iron out the last remaining bugs?