Bug Tracker

Ticket #1004 (closed bug: worksforme)

Opened 8 years ago

Last modified 4 years ago

min-height causes abnormal slide effect

Reported by: Yansky Owned by:
Priority: minor Milestone: 1.1.3
Component: effects Version: 1.1.2
Keywords: css slide fx Cc:
Blocking: Blocked by:

Description (last modified by davidserduke) (diff)

When you set a min-height for a div in your stylesheet, the  slide effects display the div all at once rather than in a smooth motion.

Example:  http://34r34r.dreamhosters.com/newdesign/new-slide-test.html

Change History

comment:1 follow-up: ↓ 2 Changed 8 years ago by john

That seems valid to me. Considering that a slideDown animates the height of an element. It seems like if you wanted to still animate something that stays at a particular minimum height you'd have to do what you posted to the wiki:  http://docs.jquery.com/Tutorials:Getting_Around_The_Minimum_Height_Glitch

I haven't tested this, but maybe something like:

$(...).animate({ minHeight: "show", height: "show" }, "slow");

might work

comment:2 in reply to: ↑ 1 Changed 8 years ago by Yansky

Replying to john: Yep fair enough. I was 50/50 as to whether I should open this ticket as a bug or as a feature request. :)

comment:3 Changed 7 years ago by davidserduke

  • need set to Review
  • Status changed from new to closed
  • Resolution set to worksforme
  • Description modified (diff)

The example no longer works and from the discussion it sounds like a questionable bug. Closing for now. Feel free to reopen it with a new test case and additional information if necessary.

comment:4 follow-up: ↓ 5 Changed 6 years ago by dmethvin

Dup #4415 has a patch, although I can't figure why you'd want to apply a min-height to divs and use this effect.

comment:5 in reply to: ↑ 4 Changed 6 years ago by dantman

Replying to dmethvin:

Dup #4415 has a patch, although I can't figure why you'd want to apply a min-height to divs and use this effect.

I use it at work. It's an editor, we've got a part of the system ui and custom stuff intertwined that can close, but when it is open we want it to actually show up with a proper size because otherwise users will be unable to use a tool to click on the empty inside in order to select it and put stuff inside.

comment:6 Changed 4 years ago by dtetto

Just a note that this (still extant) bug is re-reported as: #4415 (which, as dmethvin notes, has a patch), #4623, #5905, and #6618

Just addeed a new comment to #5905 about it since this one has been closed for years; but in the interest of completeness, here's a jsFiddle showing the bug still present in 1.4.4:

 http://jsfiddle.net/mLHb9/

comment:7 Changed 4 years ago by jitter

 test case with "workaround". It's up to the author to decide whether they want to have min-height fiddled with or not.

Note: See TracTickets for help on using tickets.