dify icon indicating copy to clipboard operation
dify copied to clipboard

fix: node shortcuts active in input fields

Open perzeuss opened this issue 10 months ago • 1 comments

Description

With #3390 I added shortcuts for interacting with nodes. These shortcuts are hadled even if you use them to manipulate text in inputs.

Type of Change

Please delete options that are not relevant.

  • [x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

  • [ ] Go to the workflow editor of a workflow or chatflow
  • [ ] Add a LLM node
  • [ ] Go to the instructions input, write text, select it and press delete
  • [ ] the text should be deleted from the input field, the node is still there. without this fix pressing delete would have deleted the llm node

Suggested Checklist:

  • [x] I have performed a self-review of my own code
  • [ ] I have commented my code, particularly in hard-to-understand areas
  • [x] My changes generate no new warnings
  • [x] I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods
  • [ ] optional I have made corresponding changes to the documentation
  • [ ] optional I have added tests that prove my fix is effective or that my feature works
  • [ ] optional New and existing unit tests pass locally with my changes

perzeuss avatar Apr 12 '24 13:04 perzeuss

@zxhlyh fyi

perzeuss avatar Apr 12 '24 13:04 perzeuss