Properties on slider container not updating when computed property changes about ember-cli-swiper

[Matt-Jensen]

Help me to understand your use case, because it seems to me that noSwiping and allowSwipeToPrev/allowSwipeToNext options are mutually exclusive as they naturally would conflict. Is there a situation you need to use both?

I guess what I'm getting at is that it would be easier for me to debug the issue once your example eliminates the possibility of incompatible settings.

[mattmcginnis]

Maybe I don't understand how it's intended to work. Which if any of these scenarios are valid?

{{#swiper-container
  noSwiping=true
  noSwipingClass='swiper-slide'
  allowSwipeToPrev=false
  allowSwipeToNext=false
}}

{{#swiper-container
  noSwiping=false
  noSwipingClass=null
  allowSwipeToPrev=true
  allowSwipeToNext=true
}}

{{#swiper-container
  noSwiping=false
  noSwipingClass=null
  allowSwipeToPrev=false
  allowSwipeToNext=true
}}

[Matt-Jensen]

1 and 2 just look redundant to me. Far simpler to just do {{swiper-container noSwiping=true}} and {{swiper-container noSwiping=false}} respectively.

3 noSwiping=false conflicts with allowSwipeToNext=false rather the component should be defined as:

{{swiper-container
  allowSwipeToPrev=false
  allowSwipeToNext=true}}

Since you need more fine grained control than noSwiping provides.

[mattmcginnis]

So then this scenario is not possible, correct?

Dynamically switch between these two conditions:

noSwiping is true only for "class1"
allowSwipeToNext is true (for classes other than "class1")
allowSwipeToPrev is false for all classes

noSwiping is false
allowSwipeToNext is true
allowSwipeToPrev is false

[Matt-Jensen]

While it's entirely possible to provide all those options together, all I'm saying is it doesn't make any sense to me why someone would do so when you could just do:

{{!-- Disabled --}}
{{swiper-container  allowSwipeToPrev=false  allowSwipeToNext=false}}

{{!-- Enabled --}}
{{swiper-container  allowSwipeToPrev=true  allowSwipeToNext=true}}

Whereas when you do:

{{swiper-container noSwiping=true  allowSwipeToPrev=true  allowSwipeToNext=true}}

You're basically saying: "The slider can swipe left and right... and swiping is disabled" those 2 instructions are obviously in conflict and which instruction wins is 100% up to how Swiper.js implements it and has nothing to do with this addon. This why I want us to stop using them together so we can get to the bug you've reported: that computed properties are not updating the container.

[mattmcginnis]

What I thought I could say is "Allow swiping for all classes except X". It seems this is not true.

[Matt-Jensen]

Fair enough, I'm going to close this for now. If you find that computed properties are not updating the swiper-container please feel free to reopen the issue.