search

inventaire / inventaire-client

webapp coupled to the inventaire server :books:
https://inventaire.io
48 stars 16 forks source link

item row: add shelf dots #366

Closed jum-s closed 1 year ago

jum-s commented 1 year ago

assumptions for PR review: most items belong to only one shelf

small screen view is a test, maybe it can just be removed completely on very small screen (as it could be too much info)

or it could be removed for group inv browser and shelf inv browser (so, leaving it only for user inv browser). No strong opinion

2022-11-20_14-25

I would not find an elegant way to display those dots in casacade view. So i propose to leave it this way for now

jum-s commented 1 year ago

Note for review: creating a shelves store would make sense in order to make a shelf update reactive, but i had difficulties updating only one shelf in the shelves array within getShelvesData

maxlath commented 1 year ago

Can you rebase it on master? It seems to have conflicts.

jum-s commented 1 year ago

Can you rebase it on master? It seems to have conflicts.

done (that was an unexpected lot indeed)