clearlydefined / website

Website for clearlydefined.io
https://clearlydefined.io
MIT License
26 stars 29 forks source link

Details view #231

Closed jeffmcaffer closed 6 years ago

jeffmcaffer commented 6 years ago

Need a reusable component that shows the full detail of a definition. It should optionally allow editing of the values in definition in a way that is shareable with other components. That is, the detail component described here does not "own" the definition. This component will show up in two primary places/ways

  1. as a popup/overlay on the main Definitions page when the user wants to dive into a given definition in the list
  2. as an independent page with a deep link URL allowing users to talk about definitions and reference them easily

A mockup of the page is in the jm/detailsPage branch. This is just a mockup and is not intended to be the implementation. There is tons of duplicate code etc

some key points

There are a few new components and related features that will show up here

AlexWebYourmind commented 6 years ago

Hi @jeffmcaffer (and all). We have just completed an Adobe XD prototype showing the main flow (Read / Edit / Contribute) for the Full Detail Screen (Modal view, DeepLinking version will be added soon)

The prototype is available at https://xd.adobe.com/view/1fef18d7-16f1-4264-57f3-75cb6d1cf9ac-69de/

We have a few questions around the Deeplinking version, one of which being:

@jeffmcaffer please review the prototype, @storrisi and @gianpaj will start actual implementation this week. As long as we are all happy with the protoype, we might use it as our goto document when we need to review the UX on that section.

jeffmcaffer commented 6 years ago

@AlexWebYourmind prototype looks good. There were a few things we talked about in the team call just now but overall, +1.

For deep linking

AlexWebYourmind commented 6 years ago

@jeffmcaffer thanks for the recap.

  1. Contribute Button Confirmed that the Contribute Button will be available on the Full Details PAGE and not in the OVERLAY/MODAL version (Save/Undo available anyways, most likely these 2 buttons will be disabled if no EDITS are made?)

  2. URL UI Routes 100% agree that the UI is not a REST API, and I think as well is beneficial to the user to have a URL like that.

My consideration was more general and in relation to existing Backend REST APIs that we will use to retrieve the data to display in the UI, but I double checked the service repository and we have everything we need. Ignore my previous misleading comment. Apologies.

storrisi commented 6 years ago

Hi @jeffmcaffer and @fossygirl, I would like to ask some questions about the SAVE button.

Currently, we have different SAVE buttons with different behaviours:

The questions are:

jeffmcaffer commented 6 years ago

Suggestions:

So short answer is, there should not be any Save buttons. :-)

storrisi commented 6 years ago

Makes sense. For what concerning the Share functionality on the Browse page, it could be something like this? https://ant.design/components/dropdown/#components-dropdown-demo-placement (the examples in the top left corner)

jeffmcaffer commented 6 years ago

Yup that's the idea.You might consider Bootstrap in this instance as the rest of that part of the UI is using it and this is not particularly complicated. See https://react-bootstrap.github.io/components/dropdowns/

jeffmcaffer commented 6 years ago

This seems done