AFFiNE icon indicating copy to clipboard operation
AFFiNE copied to clipboard

"Drag page to sidebar" does not work

Open PaxtonXuan opened this issue 2 years ago • 8 comments

https://github.com/toeverything/AFFiNE/assets/134894293/9fcd7280-c391-4f8e-a9c9-e1fa0b7e53b3

PaxtonXuan avatar Jul 27 '23 09:07 PaxtonXuan

I heard this feature was possible before

PaxtonXuan avatar Jul 27 '23 09:07 PaxtonXuan

As our expectation:

  1. Could drag and add to Favourites.
  2. Could adjust the order in Favourites.

houjoe0829 avatar Jul 30 '23 07:07 houjoe0829

to support sortable list, there are several ways:

  1. just like collection pages, we can save favorite pages as an array to idb
  2. add an array or related data structure to workspace to support sorted list
  3. add two fields like prevFavoritePage, nextFavoritePage to make pageMeta a linked list

not sure if there is more option we can choose. cc: @himself65

teobler avatar Aug 03 '23 10:08 teobler

I believe this should be saved in the CRDT to sync to all clients. So you might need to know how to use workspace.meta

himself65 avatar Aug 05 '23 04:08 himself65

Sure, this is what I proposed in option 2, then I will create an array in workspace.meta maybe named favoritePages

teobler avatar Aug 07 '23 01:08 teobler

@houjoe0829 Is this "drag and drop design" still effective?

image

Garfield550 avatar Sep 07 '23 07:09 Garfield550

@houjoe0829 Is this "drag and drop design" still effective?

image

Thank you for your attention. This design draft has expired and requires @xell to update the new design for dragging to the sidebar.

houjoe0829 avatar Sep 07 '23 08:09 houjoe0829

@houjoe0829 Is this "drag and drop design" still effective? image

Thank you for your attention. This design draft has expired and requires @xell to update the new design for dragging to the sidebar.

Okay, this issue is currently blocked. I will start working after the new design is completed.

Garfield550 avatar Sep 07 '23 08:09 Garfield550

Issue Status: ✅ Done

✅ Done

If the issue was a bug, it’s been fixed; if it was a missing feature, it’s been implemented.

This is an automatic reply by the bot.

affine-issue-bot[bot] avatar Mar 09 '24 10:03 affine-issue-bot[bot]