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

Fade In/Out #444

Open
linnett opened this issue Oct 13, 2016 · 8 comments
Open

Fade In/Out #444

linnett opened this issue Oct 13, 2016 · 8 comments
Assignees

Comments

@linnett
Copy link

linnett commented Oct 13, 2016

Is fading rather than sliding an option?

@nstanard
Copy link
Collaborator

Not currently. I'd argue that from a user-experience perspective fading isn't as good also. Thoughts? Why are you looking for a fade over a slide?

@linnett
Copy link
Author

linnett commented Oct 13, 2016

Interested to know your reasons behind why do you feel that it's a bad UX. Sliders like Slick use fade as an option - I think it's pretty common.

@nstanard
Copy link
Collaborator

nstanard commented Oct 13, 2016

Well, responsive sliders (IMO) should have touch (tablet) interaction. When a user uses their finger to slide to previous or next slides and the slider doesn't move left or right - that's very odd to me. I understand that touch interaction can be turned off and not every site wants it. I also understand it's a common feature among other sliders. However, even the term 'slider' implies movement.

The basic reason though for why I feel fade is bad UX comes down to the touch interaction. If I use my finger to try to navigate a slider I expect it to move left or right and not fade.

@nstanard
Copy link
Collaborator

Also... You don't need very much code to create your own pure JavaScript fadein/fadeout carousel.

Here are multiple examples that you can take a look at to get the functionality you want.

Simple fadein/fadeout carousel:

Make it have images that are responsive:

There ya go. This issue can be closed because this slider doesn't need this feature. What you want is a simple carousel.

@meandmax Close whenever you get a chance. ;)

@linnett
Copy link
Author

linnett commented Oct 14, 2016

@nstanard I'm aware of how simple and quick it is to do this but it's just nice to have 1 carousel that fits all - but thanks for the tips

@nstanard
Copy link
Collaborator

nstanard commented Oct 14, 2016

I define a slider differently in my head than a carousel/image gallery. I feel it's better to differentiate between those things rather than trying to create one project that fits all needs. @linnett

@nstanard
Copy link
Collaborator

nstanard commented Oct 14, 2016

Also, if you are using WebPack and ES2015 ( you should be 😄 ) then it's super easy to create your own extensible classes and differentiate between these types of features.

@nstanard nstanard self-assigned this Dec 13, 2016
@ghost
Copy link

ghost commented Jul 9, 2020

I think that fade would be a good addition since this package isn't mobile-only and the touch interaction can be turned off on desktop.

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

No branches or pull requests

2 participants