Bug Tracker

Ticket #1060 (closed enhancement: wontfix)

Opened 1 year ago

Last modified 1 year ago

slideIn and slideOut

Reported by: daemach Assigned to: anonymous
Type: enhancement Priority: trivial
Milestone: 1.1.3 Component: fx
Version: 1.1.2 Keywords:
Cc: Needs: Review

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

Change History

Changed 1 year ago by john

  • need set to Review
  • status changed from new to closed
  • resolution set to wontfix

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.

Note: See TracTickets for help on using tickets.