Calendar visibility

[jafl]

Fix calendar to use visibility:inherit instead of visibility:visible, for compatibility with overlays.

[juandopazo]

Do you think this should be done with a CSS class?

[jafl]

It could, but since it's a single style, I'm not sure it's worth the extra code in the CSS file :)

[juandopazo]

Deferring to @okuryu then.

[okuryu]

I think it doesn't need to have any CSS class for this.

@jafl Could you provide an example that causes problems with visibility: visible?

[jafl]

Here is an example: http://jafl.github.io/yui-modules/datetime/

Click the date field under "Date & Time Popup". Then click elsewhere.

In the debugger, you can see that the overlay container has the class "yui3-inputpopup-hidden" which sets visibility:hidden, but the calendar is still visible, because "yui3-calendar-content" has visibility:visible explicitly set.

My patch modifies that to visibility:inherit, so "yui3-calendar-content" doesn't interfere with parent settings.

[okuryu]

Thanks for the great example!

It looks that not only just the .yui3-calendar-content but the .yui3-calendar-grid has visibility:visible style. Do you think it doesn't need to change as the yui3-calendar-content?

[jafl]

My pull request covers both. I patched both _afterDateChange() and _rerenderCalendarPane().

[okuryu]

Aha, thanks. I could confirm that it works. Looks good to me.

[okuryu]

Merged into dev-master and dev-3.x. Thanks!