renderdoc icon indicating copy to clipboard operation
renderdoc copied to clipboard
verified publisher icon baldurk

Added shader viewer bookmarks and find shortcuts.

Open ldecarufel opened this issue 1 year ago • 6 comments

Added shader viewer bookmarks and find shortcuts.

  • Ctrl-F3 and Ctrl-Shift-F3 can now be used to quickly search for the word under the cursor
  • Shader Viewer bookmarks: Ctrl-F2 to add a bookmark, F2/Shift-F2 to go to next/previous bookmark.
  • Context menu for bookmark actions, plus "Clear All Bookmarks".

qrenderdoc_siDJ8mY4XV

ldecarufel avatar May 15 '24 00:05 ldecarufel

I thought about adding buttons, but it was a tiny little bit more involved (need to find proper icons for a start!), so I decided to implement the functionality first, with context menu actions, and I'll add buttons in a separate commit. Does that make sense to you?

ldecarufel avatar May 15 '24 14:05 ldecarufel

That's totally fine, I'll leave this PR for you to add to.

For icons you can use these pngs from the same icon pack as the others: bookmark_red.png [email protected]

baldurk avatar May 15 '24 14:05 baldurk

Would these icons be ok for Toggle/Prev/Next Bookmarks? book_picture book_previous book_next

Or maybe this one for Toggle Bookmark? bookmark

ldecarufel avatar May 16 '24 01:05 ldecarufel

Yeh I saw that last one for toggle bookmark but I thought the simple bookmark was clearer. I wasn't thinking you'd need a button for previous/next as the main idea would be to let people see that a bookmarking feature exists at all - similar to find previous/next that can be left to context menus or shortcuts.

baldurk avatar May 16 '24 09:05 baldurk

Added a bookmark menu: qrenderdoc_kp4fjETjJe

I created the blue bookmark icon myself and combined it with some existing icons.

ldecarufel avatar May 21 '24 22:05 ldecarufel

Now using icons from standard icon set. Now disabling next/prev/clear menu items when there are no bookmarks.

qrenderdoc_vxDhTBIVSG

ldecarufel avatar May 22 '24 19:05 ldecarufel