Co-Authored-By: Marshall Bowers <1486634+maxdeviant@users.noreply.github.com>

Co-Authored-By: Marshall Bowers <1486634+maxdeviant@users.noreply.github.com>

Fixes up tests for movement in editor/scrolling.

Co-authored-by: Antonio <antonio@zed.dev>

Co-Authored-By: Marshall Bowers <1486634+maxdeviant@users.noreply.github.com>

Fixes test test_copilot.

Co-authored-by: Antonio <antonio@zed.dev>

Co-authored-by: Antonio <antonio@zed.dev>

Fixes test test_edit_events.

Co-authored-by: Antonio <antonio@zed.dev>

It feels like every week or two someone hits this and is confused about
why the app isn't launch, been meaning to disable this in dev builds for
a while. This does mean that it'll be possible to hit the local DB lock
panic in a dev build if you run it multiple times but that is so rare
that it seems preferable

Release Notes:

- N/A

Co-authored-by: Antonio <antonio@zed.dev>

This reverts commit 0f7fc8c1a03eaec347df101ada66f28aefa3162a.

We've investigated another spurious failure, this time with test_multiple_excerpts_large_multibuffer; sadly it didn't really get us anywhere, so for now we're relaxing an assert.

Co-authored-by: Kirill <kirill@zed.dev>

Returned highlights were okay, but the test was trying to normalize the output by sorting the highlights by color. The ordering is different between gpui1 Color and gpui2 Hsla.

These tests failed due to an indefinite hang in buffer.condition in the following code:
\`\`\`rust
    let buffer = cx
        .add_model(|cx| Buffer::new(0, cx.model_id() as u64, text).with_language(language, cx));
    buffer.condition(cx, |buf, _| !buf.is_parsing()).await;
`\`\`
In both gpui1 and gpui2 \`.with_language\` spawns a task that notifies the context once it's done. The \`condition\` waits for notifications to be raised. The gist of the problem was that in gpui2, the spawned task was scheduled straight away, so we never really saw the notification with \`condition\`, causing us to wait indefinitely. This is probably a difference in test between schedulers in gpui1 and gpui2, but I kind of sidestepped the issue by spawning a condition before firing off a parsing task with \`set_language\`.

- +language_selector2
- Language Selector 2 working!
- Prevent languages showing in wrong order first
- copilot_menu2 (though only tested offling, which is insufficient)
- Dismiss tooltips at capture
- Get ChannelModal opening

[[PR Description]]

Release Notes:

-  N/A

(Otherwise they stay open when you hover over the editor, which
stops mouse move events propagating)

Co-Authored-By: Max <max@zed.dev>

Without this, hitting cmd-n on the context menu in the project browser
invokes the workspace::NewFile action instead of the project::NewFile
action. We're considering changing the behavior so that bindings with no
context can only invoke global actions.

Co-Authored-By: Max <max@zed.dev>

Co-Authored-By: Marshall Bowers <1486634+maxdeviant@users.noreply.github.com>

[no-ci]

Co-Authored-By: Marshall Bowers <1486634+maxdeviant@users.noreply.github.com>

With this fix, auto-updates should work on nightly.

Co-Authored-By: Marshall Bowers <1486634+maxdeviant@users.noreply.github.com>

This fixes a bug where it was possible to put the channel tree into a
bad state by accidentally dragging a channel into itself or one of its
descendants.

This PR extends `Button` with support for an optional icon to be
displayed next to the label.

As part of this, the functionality for displaying an icon within a
button has been factored out into an internal `ButtonIcon` component.
`ButtonIcon` is now used by both `IconButton` and `Button` to
encapsulate the concerns of an icon that is rendered within a button.

Release Notes:

- N/A