Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jumping to a symbol should place the cursor in the identifier #603

Open
fvictorio opened this issue Sep 20, 2024 · 0 comments
Open

Jumping to a symbol should place the cursor in the identifier #603

fvictorio opened this issue Sep 20, 2024 · 0 comments
Labels
status:ready This issue is ready to be worked on type:improvement Improvement existing feature

Comments

@fvictorio
Copy link
Member

In typescript, if you have this:

class VeryAdvancedScientificCalculator {
  /**
   * Returns the sum of two numbers.
   */
  public add(x: number, y: number ): number {
    return x + y
  }

  /**
   * Returns the product of two numbers.
   */
  public multiply(x: number, y: number): number {
    return x * y
  }
}

and you do ctrl + shift + o and select "multiply", the cursor is placed at the beginning of the multiply identifier.

With a similar code in solidity, on the other hand:

contract VeryAdvancedScientificCalculator {
  /// Returns the sum of two numbers.
  function add(uint x, uint y) public returns (uint) {
    return x + y;
  }

  /// Returns the product of two numbers
  function multiply(uint x, uint y) public returns (uint) {
    return x * y;
  }
}

if you go to the multiply symbol, the cursor is placed in the empty line before the natspec.

@fvictorio fvictorio added the type:improvement Improvement existing feature label Sep 20, 2024
@github-actions github-actions bot added the status:ready This issue is ready to be worked on label Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:ready This issue is ready to be worked on type:improvement Improvement existing feature
Projects
None yet
Development

No branches or pull requests

1 participant