zed icon indicating copy to clipboard operation
zed copied to clipboard

Tasks: UI Polish

Open maxbrunsfeld opened this issue 1 year ago • 3 comments

Check for existing issues

  • [X] Completed

Describe the feature

Some minor changes that we came up with to make the Tasks feature more polished and discoverable:

  • [ ] @osiewicz adds task discoverability improvements
  • [x] bug: with default settings, after running a task, nothing is focused.
    • By default, focus should not change - restore the previous focus on closing the task modal
  • [x] Add a "play" icon in the status bar that, when clicked, opens the task modal
    • This icon could have a status indicator
      • red if any open task tab failed
      • else, yellow if any open task tab is still running
      • else, green if there tasks tabs open, and they have all succeeded
      • else, no indicator if there are no open task tabs
  • [x] change the placeholder text of the task modal
    • lets de-emphasize "task name as prompt" / oneshot use case
    • Maybe just "Run a task..."
  • [x] Remove 'language extension' secondary text, instead display an icon indicating the language extension
    • new trait method on the TaskSource
  • [x] Task tabs for "run current function" should just say: "Rust: test_foo_bar", to make it easy to scan which test is running
    • use this same name in the task modal
    • same thing for "Test current crate", "cargo check current crate"

If applicable, add mockups / screenshots to help present your vision of the feature

No response

maxbrunsfeld avatar Apr 03 '24 17:04 maxbrunsfeld

Related: #9633

jansol avatar Apr 03 '24 21:04 jansol

For first point (a focus bug), the current behavior is correct; by default, we focus terminal & switch to a newly spawned task. Users have an option to not focus the terminal at all (though this also doesn't expand the panel if it's collapsed at the time of task spawning). There was a bug with status bar items that showed up when spawning new terminals (even when they were not task terminals); I've fixed that in #10275. We may want to change the default/add more variants of RevealStrategy to make the behavior customizable.

osiewicz avatar Apr 08 '24 15:04 osiewicz

Indicators right next to the code would be great Screenshot 2024-04-16 at 17 23 05 Screenshot 2024-04-16 at 17 22 56 Screenshot 2024-04-16 at 17 22 22

JustFrederik avatar Apr 16 '24 15:04 JustFrederik

The remaining open task is being done in https://github.com/zed-industries/zed/pull/11195 so closing this optimistically.

SomeoneToIgnore avatar May 01 '24 19:05 SomeoneToIgnore