Skip to content
This repository was archived by the owner on Oct 8, 2021. It is now read-only.

Fixed issue 1375#1354

Closed
BradBroulik wants to merge 2 commits into
jquery-archive:masterfrom
BradBroulik:master
Closed

Fixed issue 1375#1354
BradBroulik wants to merge 2 commits into
jquery-archive:masterfrom
BradBroulik:master

Conversation

@BradBroulik
Copy link
Copy Markdown

Fixed the dialog theme inheritance (issue 1375).

@scottjehl
Copy link
Copy Markdown

Thanks, BradBroulik!
Overall, this looks great. However, I'm seeing a small difference in the theming of the content div. It looks like the content area is not styled quite the same as our current dialogs. Any ideas?

Also, can you resubmit the controlgroup tests as a pull that we can review and merge on its own merit?

That'd be really helpful. Thanks again!

@scottjehl scottjehl closed this Jul 11, 2011
@scottjehl
Copy link
Copy Markdown

okay, not sure but it looks like the controlgroup tests were in already: https://github.com/jquery/jquery-mobile/tree/master/tests/unit/controlgroup.

Hmm. I guess we just need the dialog theming stuff then.
Thanks!

@BradBroulik
Copy link
Copy Markdown
Author

I'll take a look at the two dialogs and try to get them in sync.

Thanks,
Brad


From: scottjehl reply@reply.github.com
To: bbroulik@yahoo.com
Sent: Sunday, July 10, 2011 11:40 PM
Subject: Re: [jquery-mobile] Fixed issue 1375 (#1354)

Thanks, BradBroulik!
Overall, this looks great. However, I'm seeing a small difference in the theming of the content div. It looks like the content area is not styled quite the same as our current dialogs. Any ideas?

Also, can you resubmit the controlgroup tests as a pull that we can review and merge on its own merit?

That'd be really helpful. Thanks again!

Reply to this email directly or view it on GitHub:
#1354 (comment)

@BradBroulik
Copy link
Copy Markdown
Author

Hi Scott,

I just submitted a pull request titled "Fixed dialog theme inheritance (issue 1375)" that should fix the dialog theme inheritance. Please review when you have time: #2108

Thanks,
Brad


From: scottjehl reply@reply.github.com
To: bbroulik@yahoo.com
Sent: Sunday, July 10, 2011 11:40 PM
Subject: Re: [jquery-mobile] Fixed issue 1375 (#1354)

Thanks, BradBroulik!
Overall, this looks great. However, I'm seeing a small difference in the theming of the content div. It looks like the content area is not styled quite the same as our current dialogs. Any ideas?

Also, can you resubmit the controlgroup tests as a pull that we can review and merge on its own merit?

That'd be really helpful. Thanks again!

Reply to this email directly or view it on GitHub:
#1354 (comment)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants