fe. the z-index is not respected.

This issue was only about the zIndex, but most other settings are also affected.

CommentFileSizeAuthor
#2 js_settings_update-2839626-2.patch1.07 KBytsurk

Comments

Lesage Pieter created an issue. See original summary.

ytsurk’s picture

ytsurk
Location Switzerland
commented
Title: Controls z-index of the sticked element » Sticky.js configuration changed, or implemented wrong ..
Issue summary: View changes
StatusFileSize
new js_settings_update-2839626-2.patch1.07 KB

This modules add settings like this:

          selector: config.selector,
          top_spacing: config.top_spacing,
          bottom_spacing: config.bottom_spacing,
          class_name: config.class_name,
          wrapper_class_name: config.wrapper_class_name,
          center: config.center,
          get_width_from: config.get_width_from,
          width_from_wrapper: config.width_from_wrapper,
          responsive_width: config.responsive_width,
          z_index: config.z_index

but the JS lib expects it without _ but camelCased.

f0ns’s picture

f0ns commented

Thank you, I have tested and added your patch. A new release will be made.

f0ns’s picture

f0ns commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.