Skip to content

WIP: Added shortcuts#12

Open
dragonito wants to merge 1 commit intotolry:masterfrom
dragonito:master
Open

WIP: Added shortcuts#12
dragonito wants to merge 1 commit intotolry:masterfrom
dragonito:master

Conversation

@dragonito
Copy link
Copy Markdown
Contributor

No description provided.

@dragonito
Copy link
Copy Markdown
Contributor Author

fixes #11

Copy link
Copy Markdown
Owner

@tolry tolry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you tell a bit about this PR? Why two different types of shortcuts? What do they do exactly? Why those key combinations?

});

chrome.commands.onCommand.addListener(function(command) {
console.log('Command:', command);
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

still WIP?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, github != gitlab sorry

"default": "Ctrl+Shift+X",
"mac": "Command+Shift+X"
},
"description": "Adding mental notes bookmark",
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Adding MentalNote bookmark"

@dragonito dragonito changed the title Added shortcuts WIP: Added shortcuts Jul 14, 2017
@dragonito
Copy link
Copy Markdown
Contributor Author

I have not found a good shortcut yet.
Depending on the system you can put others.
In the Mac bspw you like to take the CMD Button (Apple Button)

(Google Translate) bg

@tolry
Copy link
Copy Markdown
Owner

tolry commented Jan 5, 2018

@dragonito would you mind rebasing this or can we scrap it?

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.

2 participants