Dynamic main bullets are not centered when loop

[fmatzy]

Check that this is really a bug

• I confirm

Reproduction link

https://codesandbox.io/s/swiper-pagination-dynamic-forked-teq17

Bug description

When dynamicBullets: true and loop: true are set, and dynamicMainBullets is set to a value greater than 1, the main bullets are not centered and move strangely. This is especially obvious when moving the page in the backwards direction.

I investigated the pagination module, and I found that the index shift caused by swiper.loopedSlides is not considered when calculating the dynamicBulletIndex.

swiper/src/modules/pagination/pagination.js

Line 110 in 5466df6

dynamicBulletIndex += current - swiper.previousIndex;

Expected Behavior

When loop: false, the main bullets will always be in the middle, and the same looks would be preferable when loop: true.

Actual Behavior

In the case of forward pagination, the main bullets will shift to the left at the end of the pagination.

In the case of backward pagination, strange movements like going forward and backward can be seen.

Swiper version

7.3.3

Platform/Target and Browser Versions

macOS 11.6.1, Chrome 96.0.4664.110

Validations

• Follow our Code of Conduct
• Read the docs.
• Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
• Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

• I'm willing to open a PR

[Amalykyarima]

has this been fixed?