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
[V4] Use API visibility for Carousel #17710
Conversation
I think it would be more revelant to see how it's work in |
@Johann-S I agree, it's not feasible to automatically unit test this. I think a visual test that listens for the |
<script type="text/javascript"> | ||
<!-- JavaScript Test --> | ||
$(function () { | ||
// Test to show that the carrousel don't slide when the current tab isn't visible |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"carousel doesn't"
Should add a note to the test, e.g.: <p>Also, the carousel shouldn't slide when its window/tab is hidden. Check the console log.</p> |
Done. |
Merged as 817d997. |
Hi,
this feature is for the carousel plugin to stop cycling when the page isn't visible
I re used your code @cvrebert to made my PR
I had a test unit but I think it's a bit hard to test that, if someone have a better idea how I can test that please let me know.
Because my test case is easy to test on a browser but difficult on Travis
X-Ref : #17706