>>> [!note] Migrated issue <!-- Drupal.org comment --> <!-- Migrated from issue #3459657. --> Reported by: [kopeboy](https://www.drupal.org/user/2618105) Related to !3 >>> <h3 id="summary-problem-motivation">Problem/Motivation</h3> <p>With Olivero theme, the classes in the HTML are there but the CSS and JS are not. </p> <p>I tried both with default config and with disabling css/js aggregation + twig development mode.</p> <p>I tested with a Views block with Table display, like suggested.</p> <p>It works with Claro instead (but I noticed sometimes the <code>(current)</code> is added twice on the same row).</p> <h4 id="summary-steps-reproduce">Steps to reproduce</h4> <ol> <li>Download Drupal core: <code>composer create drupal/recommended-project:10.3.2</code></li> <li>Download eva-3.1.0 <code>composer require drupal/eva:3.1.0</code></li> <li>Download views_mark_current-1.0.0-rc1 <code>composer require drupal/views_mark_current:1.0.0-rc1</code></li> <li>Install Drupal with the standard install profile: <code>drush site:install -y standard</code></li> <li>Install field_ui, views_ui, eva, and views_mark_current: <code>drush -y pm:install field_ui views_ui eva views_mark_current</code></li> <li>Login: <code>drush -y uli</code></li> <li>Go to <code>/admin/appearance</code> and set Olivero as the default theme if it is not already.</li> <li>Create a new view of content from <code>/admin/structure/views/add</code>. Add a new Display of type EVA, set the display's Format to Table, and add a Contextual Filter of type "Views mark current entity ID", and make sure the EVA attaches to Entity type: Content. The view should display the Title field; I added the ID field as well..</li> <li>Go to <code>/node/add/page</code> and add two pieces of content. The EVA should appear at the bottom. <ul> <li>Expected behaviour: The row representing the current page is displayed with a beige background, and has the word <code>(current)</code> next to the title.</li> <li>Actual behaviour: The row representing the current page is visible but does not have the beige background nor the <code>(current)</code> text.</li> </ul> </li> <li>Go to <code>/admin/appearance</code> and set Olivero as the default theme if it is not already.</li> <li>Go to one of the nodes you created earlier. The EVA should appear at the bottom. <ul> <li>Expected behaviour: The row representing the current page is displayed with a beige background, and has the word <code>(current)</code> next to the title.</li> <li>Actual behaviour: The row representing the current page is visible but does not have the beige background nor the <code>(current)</code> text.</li> </ul> </li> </ol> <h3 id="summary-proposed-resolution">Proposed resolution</h3> <p>Currently, the CSS highlights and the JS modifies table rows with the selector <code>table.views-view-table tr[data-views-mark-current="current-entity--row"]</code></p> <p>In some of the newer themes, the table element doesn't have the <code>views-view-table</code> class, meaning the default CSS and JS don't work.</p> <p>The selector <code>tr[data-views-mark-current="current-entity--row"]</code> should be unique enough; let's use that,</p> <h3 id="summary-remaining-tasks">Remaining tasks</h3> <ol> <li><del>Write a patch</del></li> <li><del>Review and feedback</del> - skipped, as it was created by maintainer and tests passed (lint failures were unrelated to these changes)</li> <li><del>RTBC and feedback</del> - skipped, as it was created by maintainer and tests passed (lint failures were unrelated to these changes)</li> <li><del>Commit</del></li> <li>Release</li> </ol> <h3 id="summary-ui-changes">User interface changes</h3> <p>None.</p> <h3 id="summary-api-changes">API changes</h3> <p>None.</p> <h3 id="summary-data-model-changes">Data model changes</h3> <p>None.</p>