Skip to content

feat: immagine copertina aggiunta e focus sistemato#1124

Open
giulio-red-turtle wants to merge 2 commits into
mainfrom
76313-Campo-immagine-di-copertina-per-video-e-videogallery
Open

feat: immagine copertina aggiunta e focus sistemato#1124
giulio-red-turtle wants to merge 2 commits into
mainfrom
76313-Campo-immagine-di-copertina-per-video-e-videogallery

Conversation

@giulio-red-turtle

Copy link
Copy Markdown

No description provided.

: data.url
}
controls
poster={placeholder}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

occhio che placeholder può essere null

if (!active || !wrapperRef.current) {
return;
}
const iframe = wrapperRef.current.querySelector('iframe');

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

non per forza c'è un tag <iframe>
ci può essere anche un tag

const iframe = wrapperRef.current.querySelector('iframe');
if (iframe) {
iframe.setAttribute('tabindex', '0');
iframe.focus();

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

così fa solo il focus sull'iframe sempre, e non lo fa piu sul bottone play dell'embed di youtube?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants