nspcc-dev / send-fs-neo-org

Send.NeoFS is a simple example of integration with NeoFS network via HTTP protocol
http://send.fs.neo.org/
GNU General Public License v3.0
1 stars 1 forks source link

Improve file panel #55

Closed mike-petrov closed 1 year ago

mike-petrov commented 1 year ago

closes #19

Signed-off-by: Mikhail Petrov mike@nspcc.ru

mike-petrov commented 1 year ago

Block looks too havy:

web:

image

mobile:

image

Few questions:

Example from @532910 send.vis.ee:

image
532910 commented 1 year ago

LGTM let's sort them in another order:

  1. URL:
  2. Metadata:
  3. ObjectID:
  4. ContainerID:
roman-khimov commented 1 year ago

You can show some/part/of/the/url/... in case it doesn't fit. I'm not sure how hard to do that dynamically depending on the screen width, maybe we can trim by the regular CID/OID length, those are shown as well anyway. CID/OID may have a "copy" button as well, just in case. URL/Meta/OID/CID order as @532910 suggested is also a good one.

mike-petrov commented 1 year ago

@532910 @roman-khimov I tried some variants more:

image image image image image
roman-khimov commented 1 year ago

I'd say they're all OK, but the last one looks a bit better to me. Maybe we can make a table for url/meta/cid/oid, that may look even better.

mike-petrov commented 1 year ago
image image
roman-khimov commented 1 year ago

It doesn't need a visible grid in the second case, but I'll let @532910 pick.

532910 commented 1 year ago

what it the reason to show url on the page directly? the URL is also useless word:

image

both 123.sr and Metadata are urls, hovering them I see url on the browser's bottom

532910 commented 1 year ago

image

532910 commented 1 year ago

image

mike-petrov commented 1 year ago

web:

image

mobile:

image
532910 commented 1 year ago

awesome!