Skip to content

Contraint les iframes à ne pas dépasser la largeur de l'écran#83

Merged
elegaanz merged 1 commit intoScribouilli:principalefrom
MickGe:patch-1
Jul 28, 2023
Merged

Contraint les iframes à ne pas dépasser la largeur de l'écran#83
elegaanz merged 1 commit intoScribouilli:principalefrom
MickGe:patch-1

Conversation

@MickGe
Copy link
Copy Markdown
Contributor

@MickGe MickGe commented Jul 27, 2023

Idéalement, il faut appliquer un attribut style="aspect-ratio: <width>/<height>;" dans l'iframe.

Copy link
Copy Markdown
Contributor

@elegaanz elegaanz left a comment

Choose a reason for hiding this comment

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

Ce style n'est pas utilisé pour le moment (il a été copié depuis un ensemble de règles pour « remettre à zéro » les styles), mais ça sera utile si jamais on ajoute des iframe un jour. Merci ! 💛

@elegaanz elegaanz merged commit 8a9b784 into Scribouilli:principale Jul 28, 2023
@DavidBruant
Copy link
Copy Markdown
Contributor

Je me permet de revert temporairement, le temps de mieux comprendre le problème

@DavidBruant
Copy link
Copy Markdown
Contributor

Ah flute, je voulais réouvrir cette PR, mais on peut pas
Bref

@MickGe Je ne suis pas sûr de comprendre la PR. Ce repo est celui de l'outil scribouilli où il n'y a pas d'iframe et donc, je ne vois pas trop l'intérêt du style sur les iframe ici

Une histoire que je me raconte, c'est que tu as créé un site Scribouilli avec une iframe et que tu t'intéresses au rendu des iframe dans ce site. Si c'était le cas, le repo pertinent n'est pas où nous discutons actuellement, mais le repo mimoza. mimoza est le thème Jekyll par défaut des sites Scribouilli. Et c'est dans ce thème que se décide le stye notamment des iframe des sites créé avec Scribouilli

Si je me trompe dans mon histoire, peux-tu nous dire où est l'iframe que tu souhaites styler ?

@MickGe
Copy link
Copy Markdown
Contributor Author

MickGe commented Jul 28, 2023

Aïe… Comment faire des ratures dans Scribouilli ! Désolé.

Le site sur lequel je me suis basé pour la PR est https://scribouilli.org/ pour lequel l'iframe vient de https://github.com/Scribouilli/documentation/blob/main/index.md.

Sur mobile, la page est défilable horizontalement à cause de l'iframe.

@maiwann
Copy link
Copy Markdown
Contributor

maiwann commented Jul 28, 2023

Merci @DavidBruant pour ton regard !
@MickGe est-ce que tu pourrais faire la même PR mais dans mimoza ? Comme ça ta proposition sera au bon endroit et quand on mergera, ça modifiera l'iframe sur scribouilli.org 😊

@MickGe
Copy link
Copy Markdown
Contributor Author

MickGe commented Jul 30, 2023

Encore désolé pour ma confusion.

Et merci pour cet outil, même si je ne l'utilise pas actuellement, l'initiative et le projet sont superbes !

Nouveau PR dans le dépôt qui va bien : Scribouilli/mimoza#8

@MickGe MickGe deleted the patch-1 branch July 30, 2023 14:24
@DavidBruant
Copy link
Copy Markdown
Contributor

Trop cool ! Merci beaucoup !

Et vraiment aucun problème pour la confusion
La responsabilité est d'abord de notre côté. Nous n'avons pas fait le taf de rendre l'environnement de développement (les repos, les relations entre eux) accueillant pour les nouvelle.eaux-venu.e.s et donc le genre d'erreur que tu as faite est complètement compréhensible

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.

4 participants