Problem/Motivation
This module is pointless if a user cannot add a note & without the "Add Note" link displaying, there's no way to add a note.
Steps to reproduce
- Enable the module
- Add the "create moderation notes" permission to a role on the user that you wish to create a note with.
- Edit a node & leave in draft mode
- Highlight some text in the body
- A link that says "Add Note" should display above the highlighted text
Proposed resolution
The issue seems to be related to the JS code in the `moderation_note.js` file that displays the link. This module was last updated in May 2020, a very long time ago. So there probably has been jQuery updates or browser JS changes that don't agree with the script anymore. This happened with another module that just needed tweaks to work with newer standards.
Remaining tasks
- Update the `moderation_note.js` file
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | after-patch.png | 483.3 KB | emilymathew |
| #3 | before-patch.png | 446.59 KB | emilymathew |
| #2 | 3294838-2.patch | 380 bytes | samuel.mortenson |
Comments
Comment #2
samuel.mortenson@mcannon I don't think this is JS related. What theme are you using? The standard profile switched its theme recently which renders in a way that puts the node content on top of the tooltip. The attached patch fixes that, but want to check first that this is what you were seeing.
Comment #3
emilymathew commentedI was facing the same issue. Thanks for the patch in #2. It is working, I have tested it with theme Olivero 9.4.3.
I was using this module and it was working perfectly fine with default theme Bartik in Drupal 9.3 version. But when I upgrade to Drupal 9.4, the default theme changed to Olivero and it was not working. But if I switch the theme to Bartik, then it will work.
Now the patch in #2 is working fine with new theme Olivero in Drupal 9.4.3.
Comment #4
jdearie commentedI was testing this module on a vanilla 9.4.3 site on Pantheon's free hosting setup. The Add note does not appear when using Olivero, but when using Bartik, the text does appear. I was trying to figure out how to add a note when I found this issue. I love the functionality of this module and will need to test with my real site / theme.
Comment #5
emilymathew commentedI missed to update the status after testing and updating in #3.
The patch in #2 works fine with Olivero theme.
Comment #7
samuel.mortensonComment #8
samuel.mortenson