Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.In mysite.com/node in latest article list links add to queue/remove from queue works as normal links (click to link fire page refresh).
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | nodequeue-non_functional_ajax_links-1965376.patch | 440 bytes | havran |
Comments
Comment #1
havran CreditAttribution: havran commentedThis patch work for me.
Comment #2
yched CreditAttribution: yched commentednodequeue.js looks quite outdated (the links behavior should be attached using Drupal.behaviors, use jquery once() insetad of manual *-processed classes, there are a couple unused vars...)
But the patch here does work as a quickfix.
Comment #3
fizk CreditAttribution: fizk commentedWhat are the steps to reproduce this?
Comment #4
fizk CreditAttribution: fizk commentedComment #5
yched CreditAttribution: yched commented@fizk:
On node/[nid], in the "links" section of the node template, nodequeue adds links to add/remove the node from the queue.
Those links are supposed to be ajax links (add/remove the node in an Ajax request without redirecting the whole page) - see the supporting code in nodequeue.js.
But that JS code is outdated and doesn't actually work, so the links act as "regular links" and redirect the browser.
The fix is quite simple, see the patch.
That's a fairly big UX regression, those ajax links on nodes are a major interaction point fro adding content to the queues.
Comment #6
yched CreditAttribution: yched commentedBack to RTBC
Comment #8
fizk CreditAttribution: fizk commentedExcellent. Committed, thanks!