Minor bug re: embedding Jitsi meeting in <iframe>

In the Jitsi Meet interface, the settings menu item “<> Embed meeting” brings up a dialog containing this html fragment:

<iframe allow="camera; microphone; display-capture" 
src="https://meet.MyDomain.com/test-002allowfullscreen="true" 
style="height: 100%; width: 100%; border: 0px;"></iframe>

Naturally this won’t work because the URL with the appended “allowfullscreen” is invalid.

The issue seems to originate in “…/react/features/embed-meeting/components/EmbedMeetingDialog.js”:

function EmbedMeeting({ t, url }: Props) {
    ...
    const getEmbedCode = () =>
        `<iframe allow="..." src="${url}`
        + 'allowfullscreen="true" ... ></iframe>';

Inserting a space as in ’ allowfullscreen=“true”’ should fix it.

Yep this was fixed last week in latest master.

Excellent! And I’m keeping a good thought that all Jitsi bugs are fixed a week before anyone notices 'em… :slight_smile: