Make .show() element-aware

[john]

From here: ​http://www.nabble.com/%24.show%28%29-improvement-tf3228045.html

the jquery function show() is at the moment making objects visible by running elm.style.display = 'block';

This works just fine for all block elements (div, etc.) but causes some problems when running on inline elements like links or other elements like table-cells.

I think it'd be better to do this:

elm.style.display = get_type(elm);

function get_type(elm) {

obj = document.createElement(elm.tagName); $('body').append(obj); type = $.curCSS(obj, 'display'); $(obj).remove(); return type;

}

This works for divs as well as for cells.

Benjamin Wiegand

[Dave]

Just creating a similar tag breaks for elements that change display type based on the cascade or a style, for example:

code {
  display: inline;
}
code.chunk {
  display: block;
}

That particular example could be fixed by using clone but the general case can't.

Doesn't setting el.style.display="" revert to the default display type? I think currentStyle/getComputedStyle return the actual display type when you do that, but I haven't tested it for a while and don't know what Safari would do with it.

[BenjaminWieg]

Doesn't setting el.style.display="" revert to the default display type?

el.style.display = "" doesn't set display to the standard display but to the display defined in the CSS (which may be also "none")

[brandon]

Patch

[genezys]

Sorry to spam but I do really think that this patch is overkill.

Just calling removeAttribute("display") on the element will just revert the display back to its default value (set in CSS by the designer). I really think that there is no issue with that and that this behaviour is better than the one in the patch.

If a <div> element is defined "inline" in the CSS, a hide()/show() should keep this setting and not revert the div to "block" just because this is the default display for a div.

[john]

Marked #1208 as a duplicate of this bug (some good test cases were provided with it).

[brandon]

Replying to genezys:

First we remove the display style property and then and only then if the element is still hidden do we try and figure out its default display style vs just using block.

[brandon]

Fixed in Rev [4074]