dotnet/roslyn

Undo should not result in a "rename" tracking session

Ouverte

#26 673 ouverte le 7 mai 2018

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)C# (4 257 forks)batch import
Area-IDEBugFeature - Renamehelp wanted

Métriques du dépôt

Stars
 (20 414 étoiles)
Métriques de merge PR
 (Merge moyen 6j 17h) (256 PRs mergées en 30 j)

Description

Due to perf issues of bringing up the rename prompt, I've been making use of renaming tracking feature to rename symbols because it's much faster. However, I keep getting into states where it gets "confused" and cannot get it to auto rename to the name I want. Here's an example:

Steps to Reproduce:

  1. Using editor, rename Class -> NewClass
    public class Class
    {
    }

Should look like this: image

  1. Press ESC (or start typing) to dismiss renaming tracking
  2. Press CTRL+Z to undo rename
  3. Rename Class -> NewerClass
  4. Drop down lightbulb

Expected Behavior: To get code fix to rename Class to NewerClass

Actual Behavior: Get a code fix that asks me to rename NewClass to NewerClass

image

Guide contributeur

Direction de recherche
Examinez comment l'opération d'annulation affecte la session de suivi de renommage. Concentrez vous sur le code qui gère l'état de la session et la pile d'annulation. Suivez les étapes du problème pour comprendre le comportement attendu.
Stack technique
csharp
Domaine
developer experience
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
45