Skip to content

fix(links): open links on click and link bubble on hover - #9202

Open
mejo- wants to merge 5 commits into
mainfrom
fix/open_link_on_click
Open

mejo- wants to merge 5 commits into
mainfrom
fix/open_link_on_click

Conversation

@mejo-

@mejo- mejo- commented Sep 14, 2026

Copy link
Copy Markdown
Member

馃摑 Summary

  • Open links straight away on click
  • On desktop open link bubble on hover
  • Add edit button to open link bubble after each link

Fixes: nextcloud/collectives#1227
Fixes: nextcloud/collectives#2713


Also fixes a11y issue with keyboard navigation: with focussed link, the Enter key now opens the link. The link bubble can be opened by, focussing the edit button behind the link and pressing Enter/Space.

Fixes: #5528


fix(linkBubble): focus href input field after Ctrl-K

Fixes: #7600


fix(LinkBubble): set tippy root position to absolute

Take the link bubble out of the flow before popper positions it, otherwise it briefly narrows the editor content and scroll anchoring scrolls the page.

Fixes: #9110

Assisted-by: OpenCode:claude-fable-5

Screenshots

Before After
image image
image image

馃弫 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests

馃 AI (if applicable)

  • The content of this PR was partly or fully generated using AI tools
  • The AI-generated content was reviewed, comprehended and tested by a human

@mejo- mejo- self-assigned this Sep 14, 2026
@mejo- mejo- added bug Something isn't working 3. to review feature: links labels Sep 14, 2026
@github-project-automation github-project-automation Bot moved this to 馃Л Planning evaluation (don't pick) in 馃摑 Productivity team Sep 14, 2026
@mejo- mejo- moved this from 馃Л Planning evaluation (don't pick) to 馃憖 In review in 馃摑 Productivity team Sep 14, 2026
@mejo-
mejo- force-pushed the fix/open_link_on_click branch 3 times, most recently from aab5a47 to 470760a Compare September 15, 2026 10:01
* Open links straight away on click
* On desktop open link bubble on hover
* Add edit button to open link bubble after each link

Fixes: nextcloud/collectives#1227
Fixes: nextcloud/collectives#2713

---

Also fixes a11y issue with keyboard navigation: with focussed link,
the Enter key now opens the link. The link bubble can be opened by
focussing the edit button behind the link and pressing Enter/Space.

Fixes: #5528

Assisted-by: OpenCode:claude-fable-5
Signed-off-by: Jonas <jonas@freesources.org>
Fixes: #7600

Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: OpenCode:claude-fable-5
Fixes wrong bubble positioning in Collectives/Deck/etc. with Firefox.

Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: OpenCode:claude-fable-5
@mejo-
mejo- force-pushed the fix/open_link_on_click branch from 470760a to 4044c88 Compare September 15, 2026 11:08
Needed for a11y reasons now that it's impossible to navigate into the
link text with the mouse (still possible with arrow keys).

Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: OpenCode:claude-fable-5
Take the link bubble out of the flow before popper positions it,
otherwise it briefly narrows the editor content and scroll anchoring
scrolls the page.

Fixes: #9110

Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: OpenCode:claude-fable-5

@marcoambrosini marcoambrosini left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

very nice improvement!

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

Labels

Projects

Status: 馃憖 In review

2 participants