eclipse-wildwebdeveloper / eclipse-wildwebdeveloper/wildwebdeveloper

WWD shortcuts to comment or uncomment code are wrong out-of-the-box (see #532 as well)

オープン
#835 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
Java
スター
201
フォーク
76
平均マージ
4日 7時間
マージ済み PR(30日)
21

説明

There is a bug it seems (at least on Windows) with the shortcut to comment lines of code.

I didn't find the way to comment out lines of code initially when using Wild Web typescript editor. I tried CTRL+SHIFT+/ or CTRL + / but those didn't do anything.

So I found issue report #352 and tried CTRL + SHIFT + C as suggested there. It still didn't work...
Then I used mouse as suggested in #352 - right click in editor / Source / Toggle line comment and that worked. And it showed as key shortcut CTRL + SHIFT + c. (so lowercase c) It started to work only after I edited the shortcut via window preferences / Keys to CTRL + SHIFT + C (capital C).

Also CTRL+SHIFT+L (the "Show Key Assist" shortcut) doesn't show this key shortcut correctly. I can only see there "Toggle Line Comment (CTRL+/)".

Please:

  1. Make the shortcut(s) work out-of-the-box for both full-line comment/uncomment (//) and partial comment/uncomment (/**/).
  2. CTRL+SHIFT + L ("Show Key Assist") when I have focus in the editor only shows "Toggle Line Comment (CTRL+/)" which, of course, doesn't work. Please make it show correctly the available commenting/uncommenting shortcut(s) in that popup.

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

ここで説明されているエディターショートカットと Key Assist の動作から始め、想定されるバインディングを issue #352 と比較します。Windows で行全体および行の一部をコメント化/コメント解除するショートカットを確認し、エディターにフォーカスがあるときに CTRL+SHIFT+L が動作するショートカットを一覧表示することを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
typescript
領域
developer-experience, tooling
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。