opencode icon indicating copy to clipboard operation
opencode copied to clipboard

feat(session modal): introduce session bookmarking

Open spoons-and-mirrors opened this issue 3 months ago • 4 comments

Building on https://github.com/sst/opencode/pull/1821

With pins

image

Pinned sessions only

image

spoons-and-mirrors avatar Aug 12 '25 03:08 spoons-and-mirrors

lol ignore these reviews they're garbage

thdxr avatar Aug 12 '25 18:08 thdxr

love the idea of pinning sessions, but a few UX/visual nits:

  • i don't love the emojis, can we indicate pins another way?
  • that bottom menu is getting a bit crowded; can we just always show with pins at the top, do we need to toggle?

adamdotdevin avatar Aug 13 '25 01:08 adamdotdevin

yeah sorry I didn't update after my latest commit, I didn't like the pins either and moved to a dot with success color (but had some troubles with the background when highlighting, showing a black box...)

anywho, it now looks like that image

I realize that pinning is actually a miss-use of the term on my part, I was more thinking of a "bookmark" system. I feel it doesn't make sense to push the pins/bookmark at the top of the session list, but rather leave them where they are in the item list. This is why I think the bookmark view is a good addition, to filter and only show our bookmarked sessions. It feels more organized this way rather than constantly pushing things at the top when we mark them.

Example: You make a few research sessions and move on to something else. Then later, you go and bookmark those research sessions because they had insight you will need at some point, but not now. It doesn't make sense to me to push them at the top, they will just be a nuisance. Plus if you have say 10 pins, you will have to scroll to see your working sessions.

Regarding the footer being too crowded, we could maybe change:

  • n new session -> n new
  • P pinned view > P filter and maybe add 2 spaces between shortcuts: n new r rename p pin P filter or 3 n new r rename p pin P filter (I like 3 better) we could also use a warning color for the letter shortcut adding some dynamism to the footer?

The highlight issue I mentioned is this image EDIT: fixed! image

PS: maybe TAB is a better way of toggling to the filtered view?

spoons-and-mirrors avatar Aug 13 '25 01:08 spoons-and-mirrors

Hey @adamdotdevin, here's how the suggestions look like

image

or

image

spoons-and-mirrors avatar Aug 14 '25 23:08 spoons-and-mirrors