I did not dig very deep so can't describe exactly what's going on. Just found out that using this theme parts of this project https://www.drupal.org/project/geolocation do not work. There is a demo geolocation-demo/common-map-ajax-with-boundary-filter using ajax to dynamically filter nodes depending on Map-input but this does not do anything on Zircon but works on all standard themes I tried...

Issue fork zircon-2881823

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

thommyboy created an issue. See original summary.

thommyboy’s picture

thommyboy CreditAttribution: thommyboy commented
Title: Zircon is doing something that prevents other modules from doing ajax stuff » Zircon is doing something that prevents other modules like geolocation from doing ajax stuff
thommyboy’s picture

thommyboy CreditAttribution: thommyboy commented

PS: regarding not working ajax in views I found this issue for the bootstrap theme https://www.drupal.org/node/2638816 and the changes they did in the patch seem to add classes like js-view-dom-id-xyz which I found are also missing in the Zircon output!

thommyboy’s picture

thommyboy CreditAttribution: thommyboy commented

solved it by modifying views-view.html.twig

line#40 needs to be changed into:

{%
  set classes = [
    'view',
    'view-' ~ id|clean_class,
    'view-id-' ~ id,
    'view-display-id-' ~ display_id,
    dom_id ? 'js-view-dom-id-' ~ dom_id,
  ]
%}
<div{{ attributes.addClass(classes) }}>
tgoeg’s picture

tgoeg CreditAttribution: tgoeg commented
Status: Active » Reviewed & tested by the community

Yes, this fixes it and it is neither fixed in the latest release on drupal.org nor in WeebPal's version yet.

Marking it as RTBC.

Gauravmahlawat made their first commit to this issue’s fork.

Gauravmahlawat opened merge request !7

Gauravvvv’s picture

Gauravvvv CreditAttribution: Gauravvvv at OpenSense Labs commented
Status: Reviewed & tested by the community » Needs review

  • Gauravmahlawat committed c6ca14f on 8.x-1.x 
    Issue #2881823 by Gauravmahlawat: Zircon is doing something that...
Gauravvvv’s picture

Gauravvvv CreditAttribution: Gauravvvv at OpenSense Labs commented

committed & pushed c6ca14f on 8.x-1.x

Gauravvvv’s picture

Gauravvvv CreditAttribution: Gauravvvv at OpenSense Labs commented
Status: Needs review » Fixed
Gauravvvv’s picture

Gauravvvv CreditAttribution: Gauravvvv at OpenSense Labs commented
Status: Fixed » Closed (outdated)