Side navigation
#1060 closed enhancement (wontfix)
Opened March 18, 2007 08:43PM UTC
Closed March 24, 2007 06:01PM UTC
Last modified March 15, 2012 05:34PM UTC
slideIn and slideOut
Reported by: | daemach | Owned by: | |
---|---|---|---|
Priority: | trivial | Milestone: | 1.1.3 |
Component: | effects | Version: | 1.1.2 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
This is probably a milestone 2.0 thing, but would it make sense to deprecate slideUp and slideDown in favor of slideIn and slideOut? The latter would match fadeIn and fadeOut which would make the effects group more consistent. The two could be given direction parameters such as: slideIn("left|right|up|down","speed",callback function).
I've debated even bringing this up because it's such a major change, but there are some benefits. It wouldn't be hard to update my existing code by doing a global replace slideUp( with slideOut("up", and then slideOut("up",) with slideOut("up")...
FWIW...
Attachments (0)
Change History (1)
Changed March 24, 2007 06:01PM UTC by comment:1
need: | → Review |
---|---|
resolution: | → wontfix |
status: | new → closed |
I'm going to have to disagree on the naming scheme. While the naming may not be perfectly ideal, it does give the user a good idea of the expected functionality. Additionally, making API changes for the sake of making API changes isn't really on the radar right now. I'm going to mark this as "wontfix", but we can re-open it for consideration if we ever get to the point where we want to make big, sweeping, API changes.