Skip to main content

Search and Top Navigation

#4201 closed bug (notabug)

Opened February 20, 2009 09:50PM UTC

Closed February 21, 2009 03:30PM UTC

Last modified February 24, 2009 03:46PM UTC

_refreshShadow in ui.dialog.js incorrectly assumes shadow is displayed

Reported by: donschuy Owned by:
Priority: minor Milestone: 1.7
Component: ui.core Version: 1.6rc6
Keywords: Cc:
Blocked by: Blocking:
Description

Consider adding a check on this.shadow before continuing in _refreshShadow or not calling _refreshShadow if a shadow is not displayed.

_refreshShadow: function(dragging) {

if(!this.shadow) return;

Attachments (0)
Change History (2)

Changed February 21, 2009 03:30PM UTC by scottgonzalez comment:1

milestone: TBD1.7
resolution: → invalid
status: newclosed

The shadow has been removed.

Changed February 24, 2009 03:46PM UTC by ammaya comment:2

Replying to [comment:1 scott.gonzalez]:

The shadow has been removed.

The shadow still exists in 1.6rc6. Will it be removed in rc7? It is not necessary to remove the shadow functionality. Just add if(self.shadow) before any instance of self._refreshShadow(1).