Skip to content

Conversation

@mootookoi
Copy link

Satori ne gère pas width: '0px' pour les images.
Du coup, si on mettait la fontSize à 0, le logo apparaissait en gros dans l'export.
Personne ne devait faire des exports sans le texte mais c'est quand même un bug =p

Comment on lines +118 to +119
// Nécessaire pour l'export. Satori ne gère pas width: 0
display: scale.fontSize(2.6) ? 'none' : 'block',
Copy link
Owner

Choose a reason for hiding this comment

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

Bien vu pour le bug ! C'est clair que c'était pas très malin de ma part de permettre une taille de police à 0px.

C'est l'occasion de le corriger pour de bon, parce que j'ai peur qu'autrement on va introduire des abstractions qui vont continuer dans cette lignée tordue.

Ce qu'on peut faire c'est de limiter la taille de police entre 8px minimum et 24px maximum. Autrement dit 0% c'est 8px et 200% c'est 24px.

Pour ça il devrait suffire de changer fontSize dans scale :

scale.fontSize = function fontSize(factor: number) {
+  const min = 8;
+  const max = 24;
+  return scale(min + (((max - min) / 2) * theme.card.fontSize));
-  return scale(16 * factor) * theme.card.fontSize;
};

Ça fait un breaking change parce que le 0% et 200% d'avant n'auront plus la même signification. Mais c'était des usages "invalides", ça ne devrait pas poser de problème.

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