Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

select, cut, copy, paste #10

Closed
forresto opened this issue Oct 17, 2012 · 4 comments
Closed

select, cut, copy, paste #10

forresto opened this issue Oct 17, 2012 · 4 comments

Comments

@forresto
Copy link
Member

and ⌘-A, X, C, V key bindings

@cannonerd
Copy link

If I have copied an element using the copy icon, pasted it with the paste icon and then continue my life by copying something from somewhere else (in this case an attribute name from my html) When I paste with ctrl -v that string to a selector in an element, it also pastes to the graph the last item I have copied using the copy icon.

@bergie bergie closed this as completed Oct 1, 2013
@bergie bergie reopened this Oct 1, 2013
@bergie
Copy link
Contributor

bergie commented Oct 1, 2013

Another similar issue:

  • Open a graph in dataflow
  • Click Ctrl-A
  • As result all nodes will be selected. But, also the search input field will be selected (since the event didn't preventDefault)

@forresto
Copy link
Member Author

forresto commented Oct 1, 2013

Something like

document.body.addEventListener('keydown', function(e) {
  if (e.target.tagName !== 'INPUT') doKeyBindings();
}, false);

@forresto
Copy link
Member Author

forresto commented Oct 1, 2013

  // Don't keybind graph actions when could be editing text #10
  if (event.target.tagName==="TEXTAREA" || 
      event.target.tagName==="INPUT" || 
      event.target.contentEditable==="true" ){ return; }

Any other cases? That bug has been following me too long to be such a simple fix.

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

No branches or pull requests

3 participants