Skip to content

Conversation

@Caephler
Copy link
Contributor

  • Cmd+M / Ctrl+M will highlight all instances of the identifier that the cursor is currently at, after which typing a new name should simultaneously change all instances of that identifier. Pressing the Esc key will bring the editor back to having only one cursor. Alternatively, clicking anywhere on the editor will make the additional cursors disappear.

  • Whenever the cursor moves or the code changes, a faint dashed box will surround all instances of it for easier identifying of which variable you are looking at.

refactoring_demo

@coveralls
Copy link

Pull Request Test Coverage Report for Build 4122

  • 9 of 38 (23.68%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.08%) to 36.951%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/components/workspace/Editor.tsx 5 34 14.71%
Totals Coverage Status
Change from base Build 4107: -0.08%
Covered Lines: 2745
Relevant Lines: 6643

💛 - Coveralls

Copy link
Member

@martin-henz martin-henz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice job. good to see the comments and the link to the Ace editor sources.

@martin-henz martin-henz merged commit 159e319 into source-academy:sa_2021 Mar 29, 2020
btzy pushed a commit that referenced this pull request May 2, 2020
* Add Refactoring functionality: Ctrl-M/Cmd-M

* Add variable highlighting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants