Skip to content

Commit

Permalink
Accessibility: Accessible iframe names
Browse files Browse the repository at this point in the history
Solves #614 by assigning an accessible title to both iframes used in
WordPress Playground. To test, use Axe devtools as described in
#614 (comment)
  • Loading branch information
adamziel committed Oct 5, 2023
1 parent 909ba4c commit 0a30dd5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions packages/playground/remote/remote.html
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@
<body class="is-loading">
<iframe
id="wp"
title="The WordPress site"
sandbox="allow-same-origin allow-scripts allow-popups allow-forms allow-downloads"
></iframe>
<script type="module">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ const JustViewport = function LoadedViewportComponent({
return (
<div className={css.fullSize}>
<iframe
title="Playground Viewport"
title="WordPress Playground wrapper (the actual WordPress site is in another, nested iframe)"
className={css.fullSize}
ref={iframeRef}
></iframe>
Expand Down

0 comments on commit 0a30dd5

Please sign in to comment.