Rewrite channel selector to use updated message types #1345
Conversation
…o fdc3-for-web-impl-addon
|
@robmoffat I've finished cleaning up the aesthetics, so this PR is ready for you to review and merge. |
There was a problem hiding this comment.
Thanks for having a go at this @julianna-ciq. Needs a few updates for new message type names. Could also ensure typing is applied to messages sent and that you only ever reference the top-level types for the messages, rather than using the payload types directly (as their naming may be unstable.
Thanks for identifying the styling race - what do you think about ways of avoiding it without using a setTimeout
| break; | ||
| } | ||
| case "iframeResolve": { | ||
| myPort.postMessage({ |
There was a problem hiding this comment.
Init and type the message as an object, then post
| payload: s | ||
| }); | ||
|
|
||
| myPort.postMessage({ |
There was a problem hiding this comment.
Init and type message then post
|
|
||
| }; | ||
|
|
||
| parent.postMessage({ |
There was a problem hiding this comment.
Init and type message then post
|
Nah, that just means there's stuff the other end to change too :-p I wasn't saying she got it wrong, rather just highlighting the places that it will need to change as we changed the message names. |
Oh right ok. Well look, I'm on holiday for the next couple of weeks, so you've got a window to get everything straightened out before I get round to merging / search and replace. |
Co-authored-by: Kris West <[email protected]>
Co-authored-by: Kris West <[email protected]>
Co-authored-by: Kris West <[email protected]>
Co-authored-by: Kris West <[email protected]>
Co-authored-by: Kris West <[email protected]>
…O/FDC3 into fdc3-for-web-impl-addon
Co-authored-by: Kris West <[email protected]>
Co-authored-by: Kris West <[email protected]>
Co-authored-by: Kris West <[email protected]>
…O/FDC3 into fdc3-for-web-impl-addon
…or-web-impl-addon
No description provided.