Microsoft/vscode

[folding] Move line up/down should skip over folded regions or folded sections

Open

#63 972 ouverte le 29 nov. 2018

Voir sur GitHub
 (14 commentaires) (75 réactions) (1 assigné)TypeScript (10 221 forks)batch import
editor-commandseditor-foldingfeature-requesthelp wanted

Métriques du dépôt

Stars
 (74 848 stars)
Métriques de merge PR
 (Merge moyen 11h 43m) (1 000 PRs mergées en 30 j)

Description

  • VSCode Version:1.30.0-insider
  • OS Version: windows 10

I think when code is in a folded region or section it should never unfold unless explicitly commanded, so if you are moving a line using move command, the line should skip over both folded regions and folded sections Steps to Reproduce:

  1. create a folded region using #region #endregion
  2. fold some code section
  3. move a line using alt+up/down over the folded region/section vscodemovelinebug

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

Guide contributeur

Direction de recherche
Examinez l'action moveLines dans le code source de l'éditeur (probablement dans src/vs/editor/contrib/moveLines/). Tracez comment les lignes sont déplacées et où l'état de pliage est affecté. Le correctif doit ignorer les régions pliées, c'est à dire que lors du déplacement d'une ligne vers le haut/bas sur une région pliée, la ligne doit sauter de l'autre côté sans se déplier.
Stack technique
typescript
Domaine
desktop
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60