dotnet/roslyn

Visual Studio doesn't adjust XML Literal tag when changing tag name

Ouverte

#32 790 ouverte le 25 janv. 2019

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)C# (4 257 forks)batch import
Area-IDELanguage-VBhelp 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

Version Used: Visual Studio 2019 Preview 2

🔗 Originally here: https://devdiv.visualstudio.com/DevDiv/_workitems/edit/132619

Steps to Reproduce:

  1. Add the following VB code:

    Module Module1

    Sub Main()
        Dim xml = <root>
                      <nam>Name</nam>
                  </root>
    End Sub

End Module

  2. Change <nam> to <name>

Expected Behavior:

</nam> changes to </name>

Actual Behavior:

The closing tag is not adjusted.

Guide contributeur

Direction de recherche
Étudiez la logique de renommage des balises littérales XML dans la couche éditeur de Roslyn. Recherchez les fichiers liés à l'édition des littéraux XML, probablement dans les fonctionnalités d'éditeur spécifiques à VB. Examinez comment les changements de nom de balise sont gérés et pourquoi la balise de fermeture n'est pas mise à jour. Considérez s'il manque une synchronisation entre les balises d'ouverture et de fermeture. Vérifiez l'historique des commits pour toute modification connexe. La correction impliquerait de mettre à jour la balise de fermeture lorsque la balise d'ouverture est renommée.
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.
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.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Visual StudioVB.NET
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70