feat(pipes): unrestrict richui iframe height and apply generous sandbox policies
This commit is contained in:
@@ -1246,11 +1246,13 @@ class Pipe:
|
|||||||
if is_html and embed_type == "richui" and rich_ui_supported:
|
if is_html and embed_type == "richui" and rich_ui_supported:
|
||||||
try:
|
try:
|
||||||
# For Rich UI Integrated view, we pass a clean iframe.
|
# For Rich UI Integrated view, we pass a clean iframe.
|
||||||
# We use 60vh directly to avoid nested iframe height collapses.
|
# We use 100vh directly to allow it to expand fully like artifacts.
|
||||||
embed_content = (
|
embed_content = (
|
||||||
f'<iframe src="{view_url}" '
|
f'<iframe src="{view_url}" '
|
||||||
f'style="width:100%; height:60vh; min-height:400px; border:none; border-radius:12px; '
|
f'sandbox="allow-scripts allow-same-origin allow-popups allow-downloads allow-forms" '
|
||||||
f'box-shadow: var(--shadow-lg);"></iframe>'
|
f'allow="fullscreen" '
|
||||||
|
f'style="width:100%; height:100vh; min-height:600px; border:none; border-radius:12px; '
|
||||||
|
f'box-shadow: 0 10px 40px rgba(0,0,0,0.1);"></iframe>'
|
||||||
)
|
)
|
||||||
|
|
||||||
if pending_embeds is not None:
|
if pending_embeds is not None:
|
||||||
|
|||||||
Reference in New Issue
Block a user