coachlogix / ember-paper-swiper Goto Github PK

I just noticed that #6 was merged but never published to npm, so octane projects would face this issue because of no jquery by default

Images are not displaying.

hi.. the demo no longer working.. so are the code we downloaded
any update to fix it?

Hi, I have attempted to implement the swiper demo into my project (https://coachlogix.github.io/ember-paper-swiper/) and encountered a bug where the width of the slides when the dialog is first opened, do not have a width that matches the width of the dialog.

This bug does not occur in the demo url above.

I'm using ember-cli: 3.11, ember-paper-swiper: 0.0.4

My demo code is a copy-paste of the example:

{{#if showSwiper}}
  {{#paper-swiper as |swiper|}}
   ....
{/if}}

I copied the CSS styles from the demo url above into my project to get the same appearance for the slides.

Please see this basic repo I created which displays the problem here

It looks like the updateContainerWidth() is not triggered correctly when the dialog is first opened. (update: I added a 1sec delay to the debounce and this shows that the dialog loads up showing 1 slide correctly, then when updateContainerWidth() is triggered from didInsertElement, it resizes the dialog width larger to show both slides at the same time. Still investigating what is going on here)

If I resize the browser window after the dialog is opened, then the slides are re-sized corrected and everything looks great.

I noticed the inline style of the slide component is controlled by this code:

widthStyle: computed('containerWidth', function() {
    let containerWidth = this.get('containerWidth');
    return containerWidth ? htmlSafe(`width: ${containerWidth}px`) : null;
}),

Example of problem:

Any help appreciated, this is a nice component. Thanks.

Will it be against mateterial design spec?
Would be nice if we can used a swiping ui inside any template.

hi ,i want to use this component without paper-wormhole,
i just dell this line "destinationId: 'paper-wormhole'," in paper-swiper.js,
can you just add this usecase as an option ?

By material design spec button to first slides need to be - Next - Next.