Skip to content
This repository was archived by the owner on Aug 12, 2018. It is now read-only.

Change shortcuts for line comment, stream comment, reset zoom and transparent mode#197

Open
popov895 wants to merge 2 commits intoXhmikosR:masterfrom
popov895:change-shortcuts
Open

Change shortcuts for line comment, stream comment, reset zoom and transparent mode#197
popov895 wants to merge 2 commits intoXhmikosR:masterfrom
popov895:change-shortcuts

Conversation

@popov895
Copy link
Copy Markdown

@popov895 popov895 commented Jul 8, 2017

Change shortcuts for the following actions:

  • Line Comment : Ctrl+Q change to Ctrl+/
  • Stream Comment : Ctrl+Shift+Q change to Ctrl+Shift+/
  • Reset Zoom: Ctrl+/ change to Ctrl+0
  • Transparent Mode: Ctrl+0 change to nothing (I guess that nobody uses this action).

I believe that new shortcuts are more convenient and more widely used, aren't they?

@XhmikosR
Copy link
Copy Markdown
Owner

XhmikosR commented Aug 6, 2017

Sorry, I don't think they are.

@XhmikosR XhmikosR closed this Aug 6, 2017
@zufuliu
Copy link
Copy Markdown
Contributor

zufuliu commented Aug 6, 2017

@XhmikosR Ctrl+/ for line comment is used by Eclipse, IntelliJ IDEA / Android Studio, NetBeans and XCode (Command+/) in default settings.

@XhmikosR
Copy link
Copy Markdown
Owner

XhmikosR commented Aug 6, 2017

Sure, but for my use case Ctrl+Q is standard and I'm used to it.

@XhmikosR
Copy link
Copy Markdown
Owner

XhmikosR commented Aug 6, 2017

That being said, if more people want this, I can always revisit. I'm mostly interested about the comment shortcuts which have been like this for years. The others are OK.

@XhmikosR XhmikosR reopened this Aug 6, 2017
@franzalex
Copy link
Copy Markdown

@popov895 I use the transparent mode regularly and I often activate it using the shortcut too. I'm quite sure I'm not the only one who does his either.
Is it possible that instead of removing the key binding, you remap it to something else?

  • Transparent Mode: Ctrl+0 change to nothing (I guess that nobody uses this action).

source

@klonos
Copy link
Copy Markdown

klonos commented Aug 6, 2017

Most applications I use that allow zooming use Ctrl++, Ctrl+-, and Ctrl+0 for zoom in, out, reset. I think that this is sort of an "industry standard". I'm talking most browsers, most photo editing apps etc.

RaiKoHoff added a commit to RaiKoHoff/Notepad3 that referenced this pull request Aug 6, 2017
- fix exception - reported in notepad2-mod pull request (Crash fixing. rizonesoft#172), which has been rejected there,
  but I was able to reproduce it in VS2017 debug mode (initial defaults for exception handling).
  (XhmikosR/notepad2-mod#172)

- incorporate changes from  notepad2-mod from v.4.2.25.995  to  v.4.2.25.998
  (one fix,  the other changes we have done before already)

- Change shortcuts for the following actions:
  (-) Toggle Transparent Mode:  moved from Ctrl+0  to  Ctrl+Numpad_*
  (x) Reset Zoom: Ctrl+/ change to Ctrl+0      ($v_5.0.26)
  Additional Accelerator Keys, as suggested by (XhmikosR/notepad2-mod#197)
  (+) Toggle Line Comment : in addition to Ctrl+Q add Ctrl+/  with same functionality
  (+) Do Stream Comment : in addition to Ctrl+Shift+Q add Ctrl+Shift+/ with same functionality
  believing new shortcuts are more convenient and more widely used, aren't they?
RaiKoHoff added a commit to RaiKoHoff/notepad2-mod-crypto that referenced this pull request Aug 6, 2017
- fix exception - reported in notepad2-mod pull request (Crash fixing. XhmikosR#172), which has been rejected there,
  but I was able to reproduce it in VS2017 debug mode (initial defaults for exception handling).
  (XhmikosR#172)

- incorporate changes from  notepad2-mod from v.4.2.25.995  to  v.4.2.25.998
  (one fix,  the other changes we have done before already)

- Change shortcuts for the following actions:
  (-) Toggle Transparent Mode:  moved from Ctrl+0  to  Ctrl+Numpad_*
  (x) Reset Zoom: Ctrl+/ change to Ctrl+0      ($v_5.0.26)
  Additional Accelerator Keys, as suggested by (XhmikosR#197)
  (+) Toggle Line Comment : in addition to Ctrl+Q add Ctrl+/  with same functionality
  (+) Do Stream Comment : in addition to Ctrl+Shift+Q add Ctrl+Shift+/ with same functionality
  believing new shortcuts are more convenient and more widely used, aren't they?
@franzalex
Copy link
Copy Markdown

franzalex commented Aug 6, 2017 via email

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants