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.
Diff assumes that the HTML ID of a node is node-NID
, which is how it addresses the node as an AJAX wrapper for handling "inline diff" functionality.
But the Omega theme does this:
function omega_alpha_preprocess_node(&$vars) {
// Prepare the arrays to handle the classes and ids for the node container.
$vars['attributes_array']['id'] = drupal_html_id('node-' . $vars['type'] . '-' . $vars['nid']);
// ....
}
If the node type is removed from the HTML ID (is there a reason why Omega includes the node type?), then inline diffs will work again.
(This is related to the Diff issue #1310422: Inline diff not rendered. The fix in that issue won't work for sites using Omega)
Comment | File | Size | Author |
---|---|---|---|
#2 | omega-2033239-2.patch | 598 bytes | pjcdawkins |
#1 | omega-2033239.patch | 560 bytes | pjcdawkins |
Comments
Comment #1
pjcdawkins CreditAttribution: pjcdawkins commentedComment #2
pjcdawkins CreditAttribution: pjcdawkins commentedActually it looks like the theme layer does need to set the node ID, but it shouldn't be including the node type.
Comment #2.0
pjcdawkins CreditAttribution: pjcdawkins commentedUpdated issue summary.