Issue Station
Sign In
Tools hereToolbox

dify - ✅(Solved) Fix improve note node [1 pull requests, 1 participants]

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…
GitHub stats
langgenius/dify#35472•Fetched 2026-04-22 08:04:00
View on GitHub
Comments
0
Participants
1
Timeline
2
Reactions
1
Author
hjlarryhjlarry
Participants
hjlarryhjlarry
Timeline (top)
cross-referenced ×1labeled ×1

Fix Action

Fixed

PR fix notes

PR #35461: fix: improve note node

Description (problem / solution / changelog)

[!IMPORTANT]

  1. Make sure you have read our contribution guidelines
  2. Ensure there is an associated issue and you have been assigned to it
  3. Use the correct syntax to link this PR: Fixes #<issue number>.

Summary

<!-- Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change. --> <!-- If this PR was created by an automated agent, add `From <Tool Name>` as the final line of the description. Example: `From Codex`. -->

fix https://github.com/langgenius/dify/issues/35472

Screenshots

BeforeAfter
......

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran make lint && make type-check (backend) and cd web && pnpm exec vp staged (frontend) to appease the lint gods

Changed files

  • web/app/components/app-sidebar/index.tsx (modified, +12/-0)
  • web/app/components/workflow/hooks/__tests__/use-nodes-interactions.spec.ts (modified, +36/-0)
  • web/app/components/workflow/hooks/use-nodes-interactions.ts (modified, +9/-1)
  • web/app/components/workflow/note-node/__tests__/hooks.spec.tsx (modified, +3/-0)
  • web/app/components/workflow/note-node/constants.ts (modified, +1/-0)
  • web/app/components/workflow/note-node/hooks.ts (modified, +2/-0)
  • web/app/components/workflow/note-node/note-editor/__tests__/editor.spec.tsx (modified, +36/-0)
  • web/app/components/workflow/note-node/note-editor/plugins/link-editor-plugin/__tests__/component.spec.tsx (modified, +54/-1)
  • web/app/components/workflow/note-node/note-editor/plugins/link-editor-plugin/__tests__/hooks.spec.tsx (modified, +25/-1)
  • web/app/components/workflow/note-node/note-editor/plugins/link-editor-plugin/component.tsx (modified, +26/-3)
  • web/app/components/workflow/note-node/note-editor/plugins/link-editor-plugin/hooks.ts (modified, +32/-3)
  • web/app/components/workflow/note-node/note-editor/store.ts (modified, +6/-1)
  • web/app/components/workflow/note-node/note-editor/theme/index.ts (modified, +1/-1)
  • web/app/components/workflow/note-node/note-editor/theme/theme.css (modified, +9/-2)
  • web/app/components/workflow/operator/hooks.ts (modified, +5/-2)
RAW_BUFFERClick to expand / collapse

Self Checks

  • I have read the Contributing Guide and Language Policy.
  • I have searched for existing issues search for existing issues, including closed ones.
  • I confirm that I am using English to submit this report, otherwise it will be closed.
  • Please do not modify this template :) and fill in all the required fields.

1. Is this request related to a challenge you're experiencing? Tell me about your story.

<img width="892" height="342" alt="Image" src="https://github.com/user-attachments/assets/056d0fc2-84a0-4c96-9dfe-aa9898aa369c" />

there are 4 issues:

  1. a link style is same as text, the users don't know this is a link
  2. to open a link, you need 3 clicks. first click will open the note edit panel, second click will open the link edit panel, then the third click can open the link. I think the first click should not occur.
  3. when selected some text then use control+b to add bold effect, it will also collapse the sidebar
  4. when enter a link, you can't use esc or click outside to cancel the enter

2. Additional context or comments

No response

3. Can you help us with this feature?

  • I am interested in contributing to this feature.

extent analysis

TL;DR

The user is experiencing issues with link styling, link opening, text formatting, and link entry cancellation in the application, and a fix would likely involve modifying the CSS, JavaScript, and keyboard event handling.

Guidance

  • Review the CSS styles applied to links to differentiate them from regular text, potentially adding an underline or changing the color to make links more distinguishable.
  • Investigate the click event handlers for links to prevent the unnecessary opening of the note edit panel on the first click, possibly by adding a conditional check to bypass this step.
  • Examine the keyboard event handling for the control+b shortcut to prevent the sidebar from collapsing when applying bold text, potentially by adding a check for the current focus element.
  • Modify the link entry functionality to allow cancellation using the esc key or clicking outside the link entry field, possibly by adding event listeners for these actions.

Example

// Example of how to add an event listener to cancel link entry on esc key press
document.addEventListener('keydown', (e) => {
  if (e.key === 'Escape') {
    // Cancel link entry logic here
  }
});

Notes

The provided information does not include specific code snippets or version numbers, so the guidance is general and may need to be adapted to the specific application codebase.

Recommendation

Apply workaround: The issues seem to be related to UI and event handling, so applying workarounds to modify the existing behavior is likely the most straightforward approach, as it does not require waiting for a new version or making significant changes to the underlying code.

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…
#API rate limit#retriever error#indexing error#inference speed#output truncation

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING

dify - ✅(Solved) Fix improve note node [1 pull requests, 1 participants]