Skip to content
This repository was archived by the owner on Mar 14, 2020. It is now read-only.

[Widget]: Add dialog support in RIB#37

Closed
xuzhang wants to merge 5 commits into
intel:masterfrom
xuzhang:dialog
Closed

[Widget]: Add dialog support in RIB#37
xuzhang wants to merge 5 commits into
intel:masterfrom
xuzhang:dialog

Conversation

@xuzhang

@xuzhang xuzhang commented Jun 5, 2012

Copy link
Copy Markdown
Contributor

Add dialog support in RIB

Comment thread src/js/views/property.js Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Better reword this:
Forbid changing the style of the first page to "Dialog"

@zhizhangchen

Copy link
Copy Markdown
Contributor

When testing, there's a problem: Uncaught SyntaxError: Duplicate data property in object literal not allowed in strict mode.
Please fix that

@xuzhang

xuzhang commented Jun 26, 2012

Copy link
Copy Markdown
Contributor Author

How to reproduce this result?

Thanks
Zhang Xu

-----Original Message-----
From: Chen, Zhang Z
[mailto:reply+i-4900477-566e49695fe57d1aa69913fbe8ae453c47e5937
8-1157089@reply.github.com]
Sent: Tuesday, June 26, 2012 4:09 PM
To: Zhang, Xu U
Subject: Re: [rib] [Widget]: Add dialog support in RIB (#37)

When testing, there's a problem: Uncaught SyntaxError: Duplicate data
property in object literal not allowed in strict mode.
Please fix that


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

@xuzhang

xuzhang commented Jun 27, 2012

Copy link
Copy Markdown
Contributor Author

The code has been rebased and basic test are done.

@xuzhang

xuzhang commented Jul 3, 2012

Copy link
Copy Markdown
Contributor Author

The code has be rebased according to latest change.

@zhizhangchen

Copy link
Copy Markdown
Contributor

Pleas rebase this pull request as it has conflicts with the master

xuzhang added 5 commits July 16, 2012 13:37
* because the first page of RIB must be a page, we don't show the
  checkbox of dialog for the first page in property view
according to style, creating page as dialog or page
In design view, we make dialog and page looked as same.
* There are different behavior on button opening target between
  JQM and RIB. For JQM, when data-role of page is "dialog" and
  the page is loaded by a button with the data-rel=page, the
  behavior of the page is "Page". Fixing this bug by modifying
  default value of opentargetas property of button.
@xuzhang

xuzhang commented Jul 16, 2012

Copy link
Copy Markdown
Contributor Author

update it.

@grgustaf

Copy link
Copy Markdown
Contributor

I went ahead and took most of this although I don't like adding more hard-coded logic about widgets in our views. That is something that will need to be a separate project to clean up.

I did not take the ddfde09 patch, see my comment.

You can close this.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants