icsharpcode/ILSpy

"override" keyword should be a hyperlink

Closed

#2,035 opened on Jun 13, 2020

View on GitHubContributor guide
 (1 comment) (0 reactions) (0 assignees)C# (3,645 forks)batch import
EnhancementHelp WantedUI

Repository metrics

Stars
 (25,162 stars)
PR merge metrics
 (Avg merge 15d 22h) (61 merged PRs in 30d)

Description

ILSpy currently has no good way of navigating to the base method of an overridden method (unless the method body contains a base-call).

In Visual Studio, executing "go to definition" on the override keyword navigates to the base method. We could do the same by making the override keyword a hyperlink.

See also: #633

Contributor guide

Research direction
Modify the decompiler's syntax highlighting to convert 'override' keyword into a clickable hyperlink that navigates to the base method. Look at existing hyperlink implementations for other keywords or identifiers.
Tech stack
csharp
Domain
frontend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
C#.NET
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60