Skip to content

fix(comment): improve comment form behavior in firefox - #7397

Merged
luka-nextcloud merged 1 commit into
mainfrom
fix-comment-form-on-firefox
Nov 27, 2025
Merged

luka-nextcloud merged 1 commit into
mainfrom
fix-comment-form-on-firefox

Conversation

@luka-nextcloud

@luka-nextcloud luka-nextcloud commented Nov 25, 2025 •

Copy link
Copy Markdown
Contributor

Summary

Fix for Firefox where clicking on/after a mention in card comment input

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Tests (unit, integration, api and/or acceptance) are included
  • Documentation (manuals or wiki) has been updated or is not required

@github-project-automation github-project-automation Bot moved this to 馃Л Planning evaluation (don't pick) in 馃摑 Productivity team Nov 25, 2025
@luka-nextcloud luka-nextcloud moved this from 馃Л Planning evaluation (don't pick) to 馃憖 In review in 馃摑 Productivity team Nov 25, 2025

@juliusknorr juliusknorr 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.

We could actually switch to https://nextcloud-vue-components.netlify.app/#/Components/NcRichContenteditable these days to not have our own at.js wrapping.

But fine with me as a bug fix 馃憤

@luka-nextcloud

Copy link
Copy Markdown
Contributor Author

@juliusknorr Sounds reasonable :) I will update the PR.

@luka-nextcloud
luka-nextcloud force-pushed the fix-comment-form-on-firefox branch from 3dc7924 to 6ebebaf Compare November 27, 2025 09:34
Comment thread css/deck.css Outdated

@juliusknorr juliusknorr 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.

Thanks for adjusting, one more commend and I think the styling also can get some cleanup. Not sure we still need the .atwho-wrap rules

Signed-off-by: Luka Trovic <luka@nextcloud.com>
@luka-nextcloud
luka-nextcloud force-pushed the fix-comment-form-on-firefox branch from 6ebebaf to eb401ff Compare November 27, 2025 10:05
@luka-nextcloud

Copy link
Copy Markdown
Contributor Author

/backport to stable32

@luka-nextcloud

Copy link
Copy Markdown
Contributor Author

/backport to stable31

@luka-nextcloud
luka-nextcloud merged commit db93f1b into main Nov 27, 2025
34 checks passed
@github-project-automation github-project-automation Bot moved this from 馃憖 In review to 鈽戯笍 Done in 馃摑 Productivity team Nov 27, 2025
@luka-nextcloud
luka-nextcloud deleted the fix-comment-form-on-firefox branch November 27, 2025 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

[Bug] comment input ignored key input after user mention

2 participants