TheWidlarzGroup/react-native-video

[BUG]: `resizeMode` prop is not implemented on web

Open

#4 297 ouverte le 21 nov. 2024

Voir sur GitHub
 (7 commentaires) (2 réactions) (0 assignés)Kotlin (3 027 forks)batch import
AcceptedMissing ReproNewer Version AvailablePlatform: WebV6bugdifficulty: easygood first issuepriority: lowtriaged

Métriques du dépôt

Stars
 (7 677 stars)
Métriques de merge PR
 (Merge moyen 2j 3h) (11 PRs mergées en 30 j)

Description

What happened?

The object-fit style property of the video element in Video.web.tsx should respect the resizeMode prop.

Currently the resizeMode prop is not implemented on web, so setting it (e.g. to "cover") has no effect on how the video is displayed.

Steps to reproduce

  1. Set resizeMode to "cover" on a Video component.
  2. Run the app on web and observe the video in a browser.
  3. Notice that the video does not respect the resizeMode value.

Reproduction repository

No response

react-native-video version

6.8.0

react-native version

No response

react-native-nitro-modules version

No response

Platforms

Web

OS version

Chrome

Device

Real device

Architecture

Old Architecture

Expo

No response

Last working version

No response

Media / source type

No response

Guide contributeur

Direction de recherche
Examinez le composant Video.web.tsx et mappez la prop resizeMode à la propriété CSS object fit sur l'élément video.
Stack technique
typescriptreact
Domaine
frontend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ReactReact NativeCSS
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70