-
Notifications
You must be signed in to change notification settings - Fork 172
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
clarify the difference between attributes and properties #3856
base: main
Are you sure you want to change the base?
Conversation
not sure if this is supposed to be in the source code somewhere. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Added snippets for the compound container, if this looks ok I can add snippets to the other API reference as well. |
```js | ||
myContainer.deferInit = true | ||
``` | ||
TODO: is it preferred to assign true or declare it to indicate it's set |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
totally fine to set it to true
``` | ||
|
||
```HTML | ||
<luigi-container viewUrl="/index.html" compoundConfig='{foo: bar}'></luigi-container> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if we should provide an example for compould attribute in html
Co-authored-by: Johannes Doberer <[email protected]>
Co-authored-by: Johannes Doberer <[email protected]>
Co-authored-by: Johannes Doberer <[email protected]>
Co-authored-by: Johannes Doberer <[email protected]>
Co-authored-by: Johannes Doberer <[email protected]>
Co-authored-by: Johannes Doberer <[email protected]>
Co-authored-by: Johannes Doberer <[email protected]>
Co-authored-by: Johannes Doberer <[email protected]>
Co-authored-by: Johannes Doberer <[email protected]>
Co-authored-by: Johannes Doberer <[email protected]>
Co-authored-by: Johannes Doberer <[email protected]>
Co-authored-by: Johannes Doberer <[email protected]>
Co-authored-by: Johannes Doberer <[email protected]>
Co-authored-by: Johannes Doberer <[email protected]>
Co-authored-by: Johannes Doberer <[email protected]>
``` | ||
|
||
```HTML | ||
<luigi-container viewUrl="/index.html"></luigi-container> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<luigi-container viewUrl="/index.html"></luigi-container> | |
<luigi-container viewurl="/index.html"></luigi-container> |
@@ -38,6 +46,83 @@ Type: [Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Globa | |||
**Meta** | |||
|
|||
* **since**: 1.0.0 | |||
TODO: better example | |||
```js | |||
myContainer.compoundConfig = lcc.compoundConfig = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
myContainer.compoundConfig = lcc.compoundConfig = { | |
myContainer.compoundConfig = { |
``` | ||
|
||
```HTML | ||
<luigi-container viewUrl="/index.html" compoundConfig='{foo: bar}'></luigi-container> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<luigi-container viewUrl="/index.html" compoundConfig='{foo: bar}'></luigi-container> | |
<luigi-container viewUrl="/index.html" compound-config='{foo: bar}'></luigi-container> |
``` | ||
TODO: is it preferred to assign true or declare it to indicate it's set | ||
```HTML | ||
<luigi-container viewUrl="/index.html" deferInit="true"></luigi-container> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<luigi-container viewUrl="/index.html" deferInit="true"></luigi-container> | |
<luigi-container viewUrl="/index.html" defer-init="true"></luigi-container> |
``` | ||
|
||
```HTML | ||
<luigi-container viewUrl="/index.html" nodeParams='{"node":"param"}'></luigi-container> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<luigi-container viewUrl="/index.html" nodeParams='{"node":"param"}'></luigi-container> | |
<luigi-container viewUrl="/index.html" node-params='{"node":"param"}'></luigi-container> |
``` | ||
|
||
```HTML | ||
<luigi-container viewUrl="/index.html" noShadow='true'></luigi-container> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<luigi-container viewUrl="/index.html" noShadow='true'></luigi-container> | |
<luigi-container viewUrl="/index.html" no-shadow='true'></luigi-container> |
``` | ||
|
||
```HTML | ||
<luigi-container viewUrl="/index.html" userSettings='{"language": "de", "theme":"sap_horizon"}'></luigi-container> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<luigi-container viewUrl="/index.html" userSettings='{"language": "de", "theme":"sap_horizon"}'></luigi-container> | |
<luigi-container viewUrl="/index.html" user-settings='{"language": "de", "theme":"sap_horizon"}'></luigi-container> |
``` | ||
|
||
```HTML | ||
<luigi-container viewUrl="/index.html" documentTitle='Luigi App'></luigi-container> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<luigi-container viewUrl="/index.html" documentTitle='Luigi App'></luigi-container> | |
<luigi-container viewUrl="/index.html" document-title='Luigi App'></luigi-container> |
``` | ||
|
||
```HTML | ||
<luigi-container viewUrl="/index.html" hasBack='/detailsPage'></luigi-container> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<luigi-container viewUrl="/index.html" hasBack='/detailsPage'></luigi-container> | |
<luigi-container viewUrl="/index.html" has-back='/detailsPage'></luigi-container> |
``` | ||
|
||
```HTML | ||
<luigi-container viewUrl="/index.html" dirtyStatus='true'></luigi-container> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<luigi-container viewUrl="/index.html" dirtyStatus='true'></luigi-container> | |
<luigi-container viewUrl="/index.html" dirty-status='true'></luigi-container> |
``` | ||
|
||
```HTML | ||
<luigi-container viewUrl="/index.html" activeFeatureToggleList='["enable-foo", "allow-bar"]'></luigi-container> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<luigi-container viewUrl="/index.html" activeFeatureToggleList='["enable-foo", "allow-bar"]'></luigi-container> | |
<luigi-container viewUrl="/index.html" active-feature-toggle-list='["enable-foo", "allow-bar"]'></luigi-container> |
Description
Changes proposed in this pull request:
Related issue(s)
Fixed #3848