Skip to content
This repository has been 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
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

popov895
Copy link

@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
Owner

XhmikosR commented Aug 6, 2017

Sorry, I don't think they are.

@XhmikosR XhmikosR closed this Aug 6, 2017
@zufuliu
Copy link
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
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
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

@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

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

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