iop
/
superset
4
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(embedded-sdk): add accessible title to iframe (#27017) 

Co-authored-by: Maxime Beauchemin <maximebeauchemin@gmail.com>
This commit is contained in:
bhaugeea 2024-06-04 17:51:14 -05:00 committed by GitHub
parent 725375541d
commit 1a52c6a3b8
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
superset-embedded-sdk/src/index.ts
View File

@ -60,6 +60,8 @@ export type EmbedDashboardParams = {
dashboardUiConfig?: UiConfigType
/** Are we in debug mode? */
debug?: boolean
/** The iframe title attribute */
iframeTitle?: string
}
export type Size = {
@ -82,7 +84,8 @@ export async function embedDashboard({
mountPoint,
fetchGuestToken,
dashboardUiConfig,
debug = false
debug = false,
iframeTitle = "Embedded Dashboard",
}: EmbedDashboardParams): Promise<EmbeddedDashboard> {
function log(...info: unknown[]) {
if (debug) {
@ -156,8 +159,8 @@ export async function embedDashboard({
// return our port from the promise
resolve(new Switchboard({ port: ourPort, name: 'superset-embedded-sdk', debug }));
});
iframe.src = `${supersetDomain}/embedded/${id}${urlParamsString}`;
iframe.title = iframeTitle;
//@ts-ignore
mountPoint.replaceChildren(iframe);
log('placed the iframe')