xtermjs/xterm.js

Support 'px' format in lineHeight option

Ouverte

#2 612 ouverte le 6 déc. 2019

Voir sur GitHubGuide contributeur
 (5 commentaires) (1 réaction) (1 personne assignée)TypeScript (1 574 forks)batch import
good first issuehelp wantedtype/enhancement

Métriques du dépôt

Stars
 (16 196 étoiles)
Métriques de merge PR
 (Merge moyen 2j 10h) (16 PRs mergées en 30 j)

Description

Now the LineHeight can only be propagated to the number type like 1.2.

There is a very uncomfortable scenario:

I chose to use DOM to render，Then the contents of the subdiv of xterm-rows, as shown below, are determined based on the configuration items, such as 1.2, and the calculated value is 21px：

image

I want to set LineHeight to 23px, then I need to set LineHeight to 1.3, but I don't know how to turn 23 to 1.3

Guide contributeur

Direction de recherche
Examinez comment lineHeight est actuellement utilisé dans le code de rendu de xterm.js, puis implémentez la prise en charge du format 'px' en analysant la valeur et en la convertissant en multiplicateur si nécessaire.
Stack technique
typescript
Domaine
frontend
Type d'issue
Fonctionnalité
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75