-
Notifications
You must be signed in to change notification settings - Fork 39
Conversation
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.
A couple of minor suggestions on the readme content. Really excited to see this land!
README.md
Outdated
|
||
The resize observer meta uses the latest [`ResizeObserver`](https://wicg.github.io/ResizeObserver/) within your widgets. This is currently supported within `Chrome 64+` and supported via [polyfill](https://github.com/WICG/ResizeObserver/issues/3). | ||
|
||
This allows you to observe resize events at the component level. The `meta` accepts an object of `predicate` functions which receive `ContentRect` dimensions and will be executed when a resize event has occured. The results are made available in your widgets `render` function. This is an incredibly powerful tool for creating responsive layouts. |
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.
in your widgets -> in a widget's
README.md
Outdated
@@ -1176,6 +1176,35 @@ class TestWidget extends WidgetBase<WidgetProperties> { | |||
} | |||
``` | |||
|
|||
#### Resize | |||
|
|||
The resize observer meta uses the latest [`ResizeObserver`](https://wicg.github.io/ResizeObserver/) within your widgets. This is currently supported within `Chrome 64+` and supported via [polyfill](https://github.com/WICG/ResizeObserver/issues/3). |
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.
within your widgets -> within Dojo-based widgets
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.
This is currently supported within Chrome 64+
and supported via -> Native browser support is currently provided by Chrome 64+
and other Dojo supported browsers work via
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.
Changes to the readme look fine. The example doesn't seem to quite work correctly on iOS, but assuming that is resolved I consider this approved.
@dylans i'll raise an issue on |
@agubler can I get some 👁 on this please? |
Type: feature
The following has been addressed in the PR:
prettier
as per the readme code style guidelinesDescription:
Creates
ResizeObserver
meta
which can be used to observe component level resize events.Requires a global polyfill to be imported when used outside of chrome 64+.
Example deployed here.
Resolves #618