Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Comment / Uncomment shortcut doesn't work on numpad #1172

Closed
brusherru opened this issue Jun 27, 2012 · 6 comments
Closed

Comment / Uncomment shortcut doesn't work on numpad #1172

brusherru opened this issue Jun 27, 2012 · 6 comments

Comments

@brusherru
Copy link

Hi!
Testing on Windows 7 x64.
This feature dowsn't work [in HTML files] with clicking in the menu, and doesn't work with pressing hotkey.
[Note: this part is a feature request, already on the backlog]

In addition, if I press CTRL+/ (on NumPad) - it opens "open file dialog".
[This is the bug being tracked here]

@warpdesign
Copy link

I guess it's not supported in every filetype.

  • html: no effect
  • js: works

Win 7 (x64)

@seanpr
Copy link

seanpr commented Jun 27, 2012

I will second this issue. Menu and shortcut do not work with HTML and CSS files, but work with JavaScript. For keyboard, I used the "/" key on my laptop's keyboard (no numpad).

Windows 7 x64
Brackets - Sprint 10

@peterflynn
Copy link
Member

Brackets doesn't support block commenting yet, which is why nothing happens in HTML and CSS files. Pull request #947 has an early implementation, and hopefully in a few weeks it'll be cleaned up enough to merge in.

Block commenting support for HTML & CSS is officially tracked by these two user stories:
https://trello.com/card/5-comment-html/4f90a6d98f77505d7940ce88/313
https://trello.com/card/5-comment-css/4f90a6d98f77505d7940ce88/312
Please feel free to vote them up if you want to give this a little extra nudge.

The NumPad issue looks like a legit bug that happens in JS files too, so I'll leave this bug open to track that issue. @brusherru, can you confirm that comment/uncomment works for you in JS content?

@ghost ghost assigned peterflynn Jun 27, 2012
@peterflynn
Copy link
Member

Updated title/description to clarify the part of this that is a bug, as opposed to a feature request. (The feature request is already tracked in the backlog stories linked above).

@RaymondLim
Copy link
Contributor

FBNC to @brusherru for using numpad issue.

@peterflynn
Copy link
Member

Confirmed fixed -- works with numpad slash now too.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants