Microsoft/vscode

Clicking inside editor and releasing the mouse button on a link outside activates the link (web/monaco)

Open

#199.477 aberto em 29 de nov. de 2023

Ver no GitHub
 (4 comments) (3 reactions) (1 assignee)TypeScript (10.221 forks)batch import
bugeditor-corehelp wanted

Métricas do repositório

Stars
 (74.848 stars)
Métricas de merge de PR
 (Mesclagem média 11h 43m) (1.000 fundiu PRs em 30d)

Description

Does this issue occur when all extensions are disabled?: Yes

  • VS Code Version: vscode.dev
  • OS Version: Win10 / Firefox

Steps to Reproduce:

  1. Go to vscode.dev
  2. Open an editor
  3. Click somewhere in the editor (e.g. to select a bunch of text) and keep the mouse button pressed
  4. Move the mouse pointer over something outside the editor, e.g. the "Read the announcement" header
  5. Release the mouse button

Nothing should happen since links/buttons are not supposed to trigger just because you release the mouse button while hovering them. After all, the click did not target that link/button.

FWIW this but is VERY annoying when embedding monaco-editor on a webapp that has a navigation near the editor's left side. When selecting a block of I almost always end up with the cursor on the menu and thus leave the page...

Guia do colaborador

Direção de pesquisa
Investigue a propagação de eventos de mouse no editor Monaco. Especificamente, verifique os manipuladores onMouseDown e onMouseUp no componente do editor e veja por que o evento mouseup em um link aciona a navegação. Veja o código de manipulação de eventos do editor em src/vs/editor/contrib/links/ ou similar.
Pilha de tecnologia
typescript
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitTypeScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65