Use Pango WrapMode in sound applet [Feature request]

[Hilyxx]

Sound applet is great but with long titles, we lost a lot of information. Could you please add (in the future) this functionality ?

Here are some mockups:

Without WrapMode

With WrapMode

Thanks

[claudiux]

Have you a piece of code to offer, please?

[Hilyxx]

@rcalixte Thank you for your commit and your reply.

I have tested your code and issue with title is always here. Sometimes titles are full displayed correctly and sometimes no. This issue occurs when sound applet is not displayed. Tested with spotify, lollypop and radio applet.

It's not a css bug, i tried with my theme and your commit in official theme. Maybe applet is not "active" in background when is not displayed and titles fails to load with WrapMode

Full title song is "It's a Long Way to the Top (If You Wanna Rock 'N' Roll")

[rcalixte]

That's interesting. I tested with some long names across several applications but I couldn't generate that. Let me test with that exact file to see if I can replicate.

One other difference is my system is in English. I wonder if that comes into play as well. I'll let you know what I can find! Thanks for the explanation!

[Hilyxx]

yes of course

File in zip, github doesn't want to share js files..
applet.zip

[rcalixte]

I moved the applet to the same position and played the audio file via VLC and Firefox.

I'm going to test with a different localization other than English. Can you try with a Mint-Y theme, using the change from the PR to remove the height property?

[Hilyxx]

This issue is hard to explain. I made a video.
While track changes, sound applet and spotify are not displayed, just in background

b.mp4

[claudiux]

Search about 'substr' in the code of this applet...

[Hilyxx]

j'ai trouvé 'substr' dans le code, je l'ai désactivé, j'ai aussi testé en augmentant la limite de caractères, le problème est toujours là. C'est vraiment quand l'applet est arrière plan.. les titres deviennent feignants.

[claudiux]

I think I have the solution... I'll post a new PR for sound150@claudiux.

[claudiux]

PR made: linuxmint/cinnamon-spices-applets#4946 and merged.
Please try sound150@claudiux v4.1.0 instead of sound@cinnamon applet. https://cinnamon-spices.linuxmint.com/applets/view/306

[Hilyxx]

Thank you . I have tried sound150@claudiux v4.1.0 and it's almost solved. I haven't issues with invisible titles. It's a great improvement .. But sometimes word wrap isn't displayed .

Original applet

sound150@claudiux v4.1.0

[claudiux]

I haven't issues with invisible titles.

Great!

But sometimes word wrap isn't displayed.

Could you be more specific? Give some examples?

[Hilyxx]

Je vais essayé d'être le plus précis possible meme si le bug en question me dépasse un peu ..

Comme je l'ai dit le problème de titres invisibles ou tronqués est je pense résolu grace à votre changement.

Par contre, et c'est encore aléatoire, le retour à la ligne des titres ne s'affiche pas toujours.

J'ai remarqué (et finalement c'était le même probleme que précédemment) que quand l'applet est affiché au premier plan (comme sur les photos) en meme temps que le changement de titre, il n'y a aucun problème.

En revanche quand l'applet n'est pas affiché et que le titre change, le retour à la ligne ne se fait pas (ou pas tout le temps) et donc il manque les derniers mots qui devraient être affichés en dessous. Que le lecteur (spotify ou autre) soit affiché ou minimisé ne change rien.

[claudiux]

Si vous aviez des exemples de titres, en précisant comment ils apparaissent (où se font les césures, où elles devraient se faire et ne se font pas), cela nous permettrait de mieux cerner le problème. De mon côté, je n'ai rien remarqué pour l'instant.

If you had some examples of titles, specifying how they appear (where the caesura occurs, where it should occur and where it doesn't), that would help us understand the problem better. For my part, I haven't noticed anything yet.

[Hilyxx]

J'ai pris pour exemple le theme officiel mint y legacy avec la police de caractère ubuntu taille 10.

Dans les exemples ci-dessous les titres sont ceux de chansons diffusées via l'applet radio. (mais c'est valable pour un lecteur lambda comme spotify ou lollypop par exemple)

La première image est celle de l'applet original sans les changements effectués de votre part. On voit que les 3/4 des mots du titre ont disparus.
Word wrap fonctionne mais aléatoirement en plus du bug de mot invisible et il faut rafraichir le code de l'applet pour que le titre apparaissent normalement. ( mais le problème réapparait juste après).

La deuxième image est celle de votre applet. Même titre. Le bug des mots invisibles a disparu mais la césure ne s'est pas faite au niveau de la fin de la ligne. Donc le mot "godfather" qui n'a pas la place pour la largeur du menu n'apparait pas et est même coupé au niveau des lettres sans retour à la ligne. (ce qui ne devrait pas arriver avec WORD_CHAR normalement). Problème aléatoire avec une césure qui peu se faire ou pas (la je ne comprends pas..). Ici je suis allé changer la radio et le titre s'est chargé (en arrière plan) AVANT que j'appuye pour afficher le menu de Sound150. et que je regarde le titre.

La troisième est toujours votre applet mais après avoir rafraichit le code via looking glass.

Deuxième exemple avec un titre différent

La première image est celle de l'applet original (la photo parle d'elle même)

La deuxième image (votre applet) montre encore un titre qui ne revient pas à la ligne avec une césure qui ne s'est pas faite à la fin du menu. Comme précédemment avec la deuxième image, je suis allé changer la radio et le titre s'est chargé (en arrière plan) AVANT que j'appuye pour afficher le menu de Sound150 et que je regarde le titre.

Pour la troisème image, je n'ai pas comme précédement rafraichi le code mais j'ai ouvert le menu de l'applet AVANT que le titre n'apparaisse. Et là aucun problème, la césure a bien a été faite au niveau des mots avec un retour à la ligne au niveau du mot "immer". (j'ai répété l'opération plusieurs fois et avec succès)

Je me suis demandé si ce n'était pas non plus un problème avec Unicode de ma LMDE5. Donc j'ai testé sur une nouvelle installation toute fraiche (linux mint 21.2) mais le problème est également présent.

[claudiux]

Odd. The ... displayed do not come from the code of the sound150 applet. You did remove the sound@cinnamon applet from the panel, didn't you?

https://gitlab.gnome.org/GNOME/pango/-/issues/243

[Hilyxx]

Non désolé j'ai oublié que les applets partagaient le même code... je viens de le refaire avec les mêmes étapes que dans mon précédent message mais le résultat est le même (sans les ... )

[claudiux]

@Hilyxx Please try to replace the line 74 of the applet.js script by this one:
const words = text.trim().replace(/[\r\n]+/g, ' ').split(' ');
(just adding .trim() after text).

Then, reload sound150. (The xsession applet can help you.)

[Hilyxx]

I tried with your commit but issue is always here..

[ghost]

@claudiux

Could this happen due to certain restricted chars not being (properly) escaped in some regular expression? Just an idea...

On another note: would there be any such fixes/improvements in your applet(s) implemented for older Cinnamon versions too, or would those be treated as obsolete? I'm asking because I plan on staying with Mint 19.2 for as long as possible, and as such would like to avoid unnecessary updates if the new code/changes wouldn't pertain to that OS version, by renaming the applet(s).

[claudiux]

I'll explore your idea, thank you.

LM 19.2 with Cinnamon 4.2? I don't know. I can try for Cinnamon 4.4 (Cinnamon 19.3) but no less.

[ghost]

Yes, it's Cinnamon 4.2.4 here.
Mint 19.3 already started downgrading/removing certain features I use daily so won't ever touch it or anything higher.
Thank you for your reply.

EDIT:
On my system I've duplicated the 3.4 folder renaming it as 4.2, and added the recent changes discussed in this topic. At first glance it works in Cinnamon 4.2. Couldn't reproduce OP's issues - maybe it only happens with ID3 text...? Tried audio files with long names and ID3s, and also video files - which included apostrophe and parenthesis - all displayed fine.
Player used was XPlayer 2.2.6, system language is English.

[claudiux]

@Hilyxx I'll keep looking, but I'm afraid I can't do much better.

[Hilyxx]

yes it's an elusive bug... :(
Thank you @claudiux for your help. I 'll keep Sound150 applet with your commits. it works fine despite the small issue with my system.

[claudiux]

It seems that version 4.1.1 of sound150@claudiux solves this problem. I will propose the same changes for [email protected].

Thanks for your help, @Hilyxx.

[Hilyxx]

yes, i have no problems with version 4.1.1 now.
Linux mint has a great team. Thank you