Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[comp:carousel] onChange的入参顺序期望是nextIndex,prevIndex #862

Closed
1 task done
brenner8023 opened this issue Apr 25, 2022 · 1 comment · Fixed by #863
Closed
1 task done

[comp:carousel] onChange的入参顺序期望是nextIndex,prevIndex #862

brenner8023 opened this issue Apr 25, 2022 · 1 comment · Fixed by #863
Assignees

Comments

@brenner8023
Copy link
Member

  • I have searched the issues of this repository and believe that this is not a duplicate.

Reproduction link

https://github.com/IDuxFE/idux/blob/main/packages/components/carousel/docs/Index.zh.md

Steps to reproduce

看文档和源码

What is expected?

onChange的入参顺序期望是nextIndex,prevIndex

What is actually happening?

onChange的入参顺序是prevIndex,nextIndex

Environment Info

Browsers:
    Chrome: 100.0.4896.127
    Edge: Spartan (44.19041.1266.0), Chromium (100.0.1185.50)
    Internet Explorer: 11.0.19041.1202
  npmPackages:
    vue: ^3.2.31 => 3.2.31
@brenner8023 brenner8023 self-assigned this Apr 25, 2022
@idux-bot
Copy link

idux-bot bot commented Apr 25, 2022

Translation of this issue:

[COMP: Carousel] Onsen's income order expects is nextindex, previndex

reproduction link

[https://github.com/iduxfe/idux/blob/main/packages/components/carousel/docs/index.zh.md]Components/Carousel/DOCS/Index.zh.md)

steps to reproduce

Look at the document and source code

What is expected?

OnChange's entry order is expected to be nextIndex, Previndex

What is actually happens?

The order of onChange is prevIDex, nextIndex

ENVIRONMENT Info

Browsers:
    Chrome: 100.0.4896.127
    Edge: Spartan (44.19041.1266.0), Chromium (100.0.1185.50)
    Internet Explorer: 11.0.19041.1202
  npmPackages:
    vue: ^3.2.31 => 3.2.31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants