Skip to content
This repository was archived by the owner on Jun 14, 2019. It is now read-only.

Modal modifiers#76

Closed
sabberworm wants to merge 4 commits intomattr-:masterfrom
sabberworm:modal-modifiers-mattr
Closed

Modal modifiers#76
sabberworm wants to merge 4 commits intomattr-:masterfrom
sabberworm:modal-modifiers-mattr

Conversation

@sabberworm
Copy link
Copy Markdown

@mattr- version of jigish#249. Allows both modal keys and the actual hotkey in modal mode to have modifiers attached. Previously only the modal mode activator key could have modifiers.

sabberworm and others added 4 commits February 20, 2013 16:25
A key bound like

	key:modal;modifier

Behaves the way it always has: the modifier is applied to the key that enters modal mode.

A key bound like

	key:modifier;modal

Now behaves as follows: the modifier has to be pressed with the key that triggers the action while in modal mode.

Of course, the two can also be combined:

	key:modifier1;modal;modifier2

behaves as you’d expect: modal mode is activated when `modal` is pressed along with `modifier2`
and the command is executed if `key` is pressed along with `modifier1` while in modal mode.

combinations followed by `:toggle` also work
* origin/master: (75 commits)
  added .DS_Store to .gitignore
  fixed issue jigish#70; undo PR 72
  Fix Issue jigish#70 Cannot resize at right border
  Fix content size mismatch warning.
  Fix a test name clash with the test framework
  Fix wrong hint icons
  Hide ghost windows of 1 by 1 pixels.
  Fix empty title window hints.
  Fix link to default config file
  Adopt JavaScript config documentation from wiki.
  Move installation and configuration documentation to separate markdown files in new doc directory; improve formatting; improve clarity.
  Do not hide windows without title from WindowInfoView
  If given an AccessibilityWrapper, use it
  Remove JSONKit usage from the project
  Fix the indentation
  replace c and missed <space> with constant
  Used a constant instead of text directly.
  fixed spaces.
  Handle space within the shell path
  Whitespace
  ...
@mattr- mattr- closed this Jun 14, 2019
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.

2 participants