Initial display of swiper contains slides with a width too short to fit dialog width

[darenyong]

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.

[darenyong]

Hi,

Well I found a fix for getting the slides to display correctly (with an inline width style that matches the max-width of md-dialog).

in container.js, I added this:

  didRender() {
    this.updateContainerWidthDebounced();
  },

That seems to ensure the slides always have their width set correctly.

Note that simply increasing the debounce time did not help.