Open-Assistant icon indicating copy to clipboard operation
Open-Assistant copied to clipboard

#309 - Message navigation

Open jojopirker opened this issue 2 years ago • 1 comments
trafficstars

Draft for an implementation for #309.

Looks like this currently: https://user-images.githubusercontent.com/19796430/210849009-df5c03d8-d67d-4646-8091-64e555d49aff.mov

jojopirker avatar Jan 06 '23 10:01 jojopirker

made a few changes to make it recursive... Looks the same if message tree is not big, but still has some visual quirks with a bigger trees (see attached video)

@fozziethebeat maybe you got some ideas how to properly style it... I would leave it as a draft still but I think you could already take a look at the code, please :)

https://user-images.githubusercontent.com/19796430/211035618-0bb5383a-9bfd-44dd-a556-1627a6b15569.mov

jojopirker avatar Jan 06 '23 14:01 jojopirker

I don't see many code suggestions to make. It's clean and pretty easy to understand. Later I think we can fold the backend fetch calls into a library someone is working on but I'm also trying to negotiate not having these API routes anyways so it's not a huge issue.

Will approve once you feel it's ready

fozziethebeat avatar Jan 06 '23 23:01 fozziethebeat

should be ready now :)

Screenshot 2023-01-07 at 10 23 59

jojopirker avatar Jan 07 '23 09:01 jojopirker

This actually looks cool!!!

AbdBarho avatar Jan 07 '23 10:01 AbdBarho