Dialog module currently includes all css files from jQuery UI, but only really needs 4 of the 12 files.
In addition, a theme or module might want to provide it's own theme css, so we need a way to not include 'ui.theme.css'.

CommentFileSizeAuthor
#3 patch_commit_2a703a20940c.patch946 bytesmoonray
#2 patch_commit_2c2e593ee4d5.patch1.51 KBmoonray
#1 patch_commit_82addcc3b11d.patch1.26 KBmoonray

Comments

moonray’s picture

moonray commented
Status: Active » Needs review
StatusFileSize
new patch_commit_82addcc3b11d.patch1.26 KB

Attached patch fixes the described issues.

moonray’s picture

moonray commented
StatusFileSize
new patch_commit_2c2e593ee4d5.patch1.51 KB

Let's try that again with proper PHP Docs.

moonray’s picture

moonray commented
StatusFileSize
new patch_commit_2a703a20940c.patch946 bytes

Hrm. After a brief dicussion with Jacine it seems we don't need to worry about not loading theme.ui.css. A theme can simply override that css file in its .info file.

devin carlson’s picture

devin carlson commented
Issue summary: View changes
Status: Needs review » Closed (outdated)

Dialog 6.x is no longer supported.