continue icon indicating copy to clipboard operation
continue copied to clipboard

First Lens.. Mouse over TODO's

Open rootedbox opened this issue 1 year ago • 3 comments

Description

Added the ability to mouse over and react to TODO's

Screen Shot 2024-04-23 at 21 13 07 PM

rootedbox avatar Apr 24 '24 04:04 rootedbox

@sestinj is the auto linter removing the last comma? see commands.ts

rootedbox avatar Apr 24 '24 04:04 rootedbox

@rootedbox Yes most likely Prettier is doing that. But Peter Zaback added "trailingComma": "all" to the .prettierrc in his recent PR, which will help standardize

sestinj avatar Apr 24 '24 07:04 sestinj

My thoughts after pulling and playing around with this a bit...I like that it's not a codelens because in a way that makes it less invasive. The one downside is that this feels like a bit of a hidden feature that would be extremely rare to discover. More importantly though, I am not sure how frequently this will be able to accomplish a TODO, which in practice is often a multi-file edit, or something difficult enough to accomplish that somebody decided to postpone the work. That is mostly feedback on the code insertion feature. As for chat, I can see more success.

Also: I'm curious what the limits of this hover view are. If there's any way to display something like a button, an image, or just larger text, this might be an extremely valuable UI tool

sestinj avatar Apr 24 '24 07:04 sestinj