Skip to content

Optional expose on right click - #76

Closed
Dravenshorst wants to merge 17 commits into
rlamana:masterfrom
Dealerdirect:disable-expose-on-right-click
Closed

Optional expose on right click#76
Dravenshorst wants to merge 17 commits into
rlamana:masterfrom
Dealerdirect:disable-expose-on-right-click

Conversation

@Dravenshorst

Copy link
Copy Markdown
Contributor

Adds the option to disable/enable the expose on right-click.

brammittendorff-dd and others added 14 commits June 13, 2018 13:10
…nimations

Added the disabling animations
…nimations

Added getters and setters and moved addclass to the constructor
* Adds new compiled js files in dist folder.
* Adds rightClick option to examples `desktop` and `simple`.
* Adds variable and a set method for rightClick in `expose.js`.
* Adds wmOptions to WindowManager constructor, and adds default options for windowManager (for now only `rightClick` in default options.

Signed-off-by: Daniel Ravenshorst <daniel.ravenshorst@dealerdirect.nl>
Updated master from Ventus
* Updates `readme.md`.

Signed-off-by: Daniel Ravenshorst <daniel.ravenshorst@dealerdirect.nl>
Signed-off-by: Daniel Ravenshorst <daniel.ravenshorst@dealerdirect.nl>

# Conflicts:
#	dist/ventus.min.js
Signed-off-by: Daniel Ravenshorst <daniel.ravenshorst@dealerdirect.nl>

# Conflicts:
#	dist/ventus.min.js
…inside-of-space

Added option to stay inside of space
Signed-off-by: Daniel Ravenshorst <daniel.ravenshorst@dealerdirect.nl>
@Dravenshorst Dravenshorst changed the title Disables expose on right click Optional expose on right click Jun 14, 2018
Signed-off-by: Daniel Ravenshorst <daniel.ravenshorst@dealerdirect.nl>
Signed-off-by: Daniel Ravenshorst <daniel.ravenshorst@dealerdirect.nl>

# Conflicts:
#	dist/ventus.min.js
Signed-off-by: Daniel Ravenshorst <daniel.ravenshorst@dealerdirect.nl>

# Conflicts:
#	dist/ventus.min.js
@rlamana

rlamana commented Jun 15, 2018

Copy link
Copy Markdown
Owner

@Dravenshorst was this closed for any particular reason?

@Dravenshorst

Copy link
Copy Markdown
Contributor Author

@rlamana, yes it had many changes from our master, forked from yours. So the branch had more changes than this pull request should have. So I have created a new one: #78

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants