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

Fix the tinyMCE mention plugin #4852

Merged
merged 2 commits into from
Jan 14, 2024

Conversation

MarcelBolten
Copy link
Contributor

It turns out this problem was encountered by others as well StevenDevooght/tinyMCE-mention#93 and a fix was provided. However, a few more changes where made in this PR.

@MarcelBolten MarcelBolten linked an issue Jan 14, 2024 that may be closed by this pull request
@NicolasCARPi
Copy link
Contributor

Seems to work fine on FF, but Chromium gives:

Uncaught TypeError: Cannot read properties of null (reading 'textContent')
    at AutoComplete.lookup (plugin.js:149:85)
    at AutoComplete.rteKeyUp (plugin.js:77:22)
    at Editor.proxy (jquery.js:10615:13)
    at EventDispatcher.dispatch (tinymce.js:30008:31)
    at Editor.dispatch (tinymce.js:30124:57)
    at fireEvent (tinymce.js:30169:16)
    at Object.delegate (tinymce.js:30214:11)
    at EventUtils.executeHandlers (tinymce.js:4169:43)
    at HTMLBodyElement.defaultNativeHandler (tinymce.js:3991:16)

@MarcelBolten
Copy link
Contributor Author

but textContent is replaced with innerText. I will need to look again.
Did you empty the browser cache?

@NicolasCARPi
Copy link
Contributor

Correct, it was a cache issue, sorry! Looks good then! :) 👏

@NicolasCARPi NicolasCARPi merged commit db3fe4f into hypernext Jan 14, 2024
8 of 9 checks passed
@NicolasCARPi NicolasCARPi deleted the marcel-fix-tinymce-mention-plugin branch January 14, 2024 23:01
NicolasCARPi added a commit that referenced this pull request Jan 16, 2024
* 'hypernext' of github.com:elabftw/elabftw:
  Fix the tinyMCE mention plugin (#4852)
NicolasCARPi added a commit that referenced this pull request Jan 20, 2024
* hypernext:
  fix required label not appearing on all elements
  Fix ove and issues with uploadsDiv and the observer (#4867)
  Avoid casting in favor of getString and get Int; harmonization of the use of App->Request (#4872)
  fix drag and drop issues when uploads in table display mode
  add sysconfig setting to prevent admins from archiving users
  add a notification when new version has been updated after login
  add configurable chat room link
  add emit_audit_logs instance config to emit audit logs as NOTICE in php
  add import/export audit log event
  try and fix issue with multiple data-trigger listeners on the page
  add requester info to Users2Teams class
  simplify function notation
  don't disable the timestamp button on click
  always display main text + extra fields in toggle body in show mode
  add custom_id to CSV export
  add owner parameter for GET all
  add diff syntax highlighting
  add `metadata_decoded` to JSON output for single entity
  Fix the tinyMCE mention plugin (#4852)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Autocomplete with # doesn't work with Chromium
2 participants