plotly / plotly/plotly.js

`responsive` doesn't work if a parent has padding and `min-width: min-content`

Ouverte
#7,491 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

bug P2
Langage dominant
JavaScript
Étoiles
18.3k
Forks
2k
Merge moyen
2 j 12 h
PR mergées (30 j)
28

Description

steps to reproduce:

  1. open this fiddle https://jsfiddle.net/rhdq9ynu/5/
  2. use ctrl+mousewheel to zoom out to 30%
  3. use ctrl+mousewheel to zoom back in to 100%
  4. notice that the plot now overflows its parent and the home icon in the top is no longer visible.

before

Image

after

Image

The key is this CSS:

#app {
  padding: 0px 100px;

  & > * {
    min-width: min-content;
  }
}

If you remove either the padding or the min-width everything works fine.

Both min-content and fit-content are buggy.

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par le JSFiddle lié et reproduisez la séquence de zoom avec le padding et le CSS min-width indiqués. Aucun fichier du dépôt ni aucun test n’est nommé ; le travail est terminé lorsque le graphique reste dans son élément parent et que l’icône d’accueil reste visible après un retour du zoom à 100 %, y compris avec min-content et fit-content.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
css, javascript
Domaine
data-visualization, frontend
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.