Skip to content
This repository has been archived by the owner on Jul 24, 2020. It is now read-only.

Modifier l'espacement entre deux lignes (text_show) #470

Open
YendaHusk opened this issue May 15, 2020 · 9 comments
Open

Modifier l'espacement entre deux lignes (text_show) #470

YendaHusk opened this issue May 15, 2020 · 9 comments
Projects

Comments

@YendaHusk
Copy link

Problème lié à une commande

Configuration (lorsque le bug a été rencontré)

  • Version de RME : 2.0.0
  • Commande : text_show
  • Autres scripts (s'il y en a) : Orms

Description du bug

Hello ! Je travail sur un système de dialogue custom via la commande text_show et text_progressive.
J'aimerais pouvoir stocké mes dialogues en une seul ligne dans un fichier texte. Ça fonctionne sans aucun souci de ce côté.
Par contre, pour faire des dialogues plus long j'aimerais utilisé le retour à la ligne. Ce que je fait via la commande \n
Le problème que je rencontre, c'est l'espacement entre les deux lignes qui est beaucoup trop grand.
Du coup je me demander si il été possible de configuré ça, ou alors de le forcer d'une manière ou d'une autre ?
image

@xvw xvw added this to Opened in 3.0.0 May 15, 2020
@BilouMaster
Copy link
Member

Ca peut être une forme de conflit entre les deux scripts (ORMS / RME, les deux manipulent la valeur d'espacement du texte) puisque l'espacement attendu devrait être égal à celui d'un texte normal (Commande événement > Message) avec ou sans ORMS.

Je suis un peu occupé là... (comme toujours) alors si tu veux tordre un peu (provisoirement) les paramètres de ORMS pour comparer la différence entre text_show et la commande événement habituelle, ça pourrait déjà t'aider à mieux préciser les paramètres qui entrent en jeux.

Sinon un petit patch et on n'en parle plus. ;)

j.🌸

@YendaHusk
Copy link
Author

"Malheureusement" le problème ne viens pas d'Orms, même sans, le décalage est toujours présent.
J'ai essayer de jouer un peu avec les valeurs de Orms, notamment FONT_HEIGHT et LINE_HEIGHT
La première ne donne évidemment rien de bon et la deuxième change effectivement les espacements entre les lignes, mais pour les menus uniquement, ça n'affecte pas les retours à la ligne. =/
image

@xvw
Copy link
Contributor

xvw commented May 15, 2020

C'est un bug. L'espacement entre les textes n'est pas calculé de la même manière que les messages.

@YendaHusk
Copy link
Author

YendaHusk commented May 15, 2020

Il y aurait un moyen de remédier à ça temporairement avec un patch ou autres ?

@BilouMaster
Copy link
Member

BilouMaster commented May 15, 2020

Bien sûr, mais @xvw semble occupé. J'allais chercher la ligne où était calculé ce fameux espacement pour comprendre sur quoi xvw s'était basé (si ce n'est pas celui par défaut, j'imagine un espacement configuré dans le profil de la font)... Mais finalement je dois reconfigurer mon PC avant, je l'ai réinitialisé il y a pas longtemps.

Si j'y pense, je verrais pour bricoler un patch demain, sinon si tu trouve toi-même, n'hésite pas à patcher toi-même et partager ici la manière dont tu as patché... C'est bon d'avoir une certaine vision de comment fonctionnent les scripts que tu ajoutes à ton projet et de savoir les patcher toi-même si besoin. (Et encore plus cool de partager tes patchs qui pourraient inspirer des corrections, ajouts ou modifications pour la prochaine MàJ si le projet est encore maintenu.)

@YendaHusk
Copy link
Author

Du coup j'ai chercher dans les 6800 lignes de codes d'rme, j'ai pas réussi à trouver (ou à comprendre) la ligne qui gère ça. Si vous savez comment la trouver je veux bien essayez par moi-même de bidouiller ça. Même si je doute d'en avoir les compétences.. :D

@gr-im
Copy link
Contributor

gr-im commented May 16, 2020

6800 ? RME fait ~17400 lignes (sans la documentation) !
La méthode qui gère les textes se trouve ici : https://github.com/RMEx/RME/blob/master/src/EvEx.rb#L3455

Je pense que ici : https://github.com/RMEx/RME/blob/master/src/EvEx.rb#L3467 :

width, height = widths.max + font.size, heights.max + font.size

le dernier +font.size semble être de trop.

Quoi qu'il en soit, je pense qu'il faudrait réécrire "un peu" le support de textes et sortir l'attribut size du profil.

@YendaHusk
Copy link
Author

YendaHusk commented May 16, 2020

Ah oui effectivement, j'avais pas fait attention qu'une fois rendu à 9999, le compteur reprenait à 0.
Du coup merci beaucoup ! Le + font.size était effectivement en trop.
J'ai également modifié la ligne du dessous pour qu'une nouvelle ligne fasse remonté le texte plutôt que d'afficher les lignes suivantes en dessous.

    width, height = widths.max + font.size, heights.max
    total_height = height * lines.length + (font.size * lines.length)

Merci de votre aide à tous ! :)
Et vive RME ! :D

@xvw
Copy link
Contributor

xvw commented May 16, 2020

Merci @pytre !
Effectivement, je corrigerai ça dans la version 2.2.0.
Je suis aussi d'accord sur le fait de retirer la taille des profiles (et d'ajouter un interligne).

@xvw xvw moved this from Opened to In progress in 3.0.0 Jul 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
No open projects
3.0.0
In progress
Development

No branches or pull requests

4 participants