Would be nice to have an include option.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Erik Seifert’s picture

Here is a patch

13rac1’s picture

Version: 7.x-1.3 » 7.x-1.x-dev
Status: Active » Needs work

Doesn't apply to current dev. Please correct.

sgurlt’s picture

Issue summary: View changes

I will work on that.

sgurlt’s picture

Assigned: Unassigned » sgurlt

  • 9b24044 committed on 7.x-1.x
    Issue #2005854 by Erik Seifert: Include / Exlude option
    
legolasbo’s picture

Status: Needs work » Fixed

I've altered the exclude option to match the include/exclude functionality provided by the block module.

legolasbo’s picture

Assigned: sgurlt » Unassigned
Status: Fixed » Needs work
Parent issue: » #2457681: Image Lazyloader 1.4 (new stable release)

We still need an upgrade path from the old way of storing settings to the system_settings_form() way of doing so. This blocks the release of 1.4

legolasbo’s picture

Priority: Normal » Critical
Issue tags: +Novice

Bumping priority to Critical because this is blocking a release. Tagged novice to inspire new contributors to take on this small problem.

  • legolasbo committed a3390dd on 7.x-1.x
    Issue #2005854 by legolasbo: Include / Exlude option
    

  • 9b24044 committed on 7.x-1.x
    Issue #2005854 by Erik Seifert: Include / Exlude option
    
  • legolasbo committed a3390dd on 7.x-1.x
    Issue #2005854 by legolasbo: Include / Exlude option
    
legolasbo’s picture

Status: Needs work » Fixed

Upgrade path added. New release pending publication! :)

Status: Fixed » Closed (fixed)

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