Author |
Topic: Nouvelle version du script myrweb.js avec Play/Repeat (Read 3774 times) |
|
JacquesF
Board Newbie
Posts: 25
|
Bonjour à tous,
Le script de Gilbert Rouquie (et les options d'André Baeck) ne fonctionnant plus avec la dernière version de HA (9.9.6c), et comme j'avais besoin des exploitations "Jouer à partir de" et "Répétition", je me suis plongé dans le code pour réintégrer les modifications.
Je ne suis pas du tout expert en javascript mais je me suis basé sur le code de Gilbert, en rendant la lecture plus facile avec quelques fonctions supplémentaires.
De plus, j'ai ajouté quelques possibilités permettant de personnaliser ce script sans avoir à modifier le code par la suite.
Pour faire simple, sans aucune option le script fonctionne comme celui de Gilbert à l'origine (avec toutefois les caractères proposés par André).
Il y a juste un message d'alerte qui survient pour interdire la sélection si le zoom est activé (le résultat n'étant pas du tout fiable).
Autrement, il est possible de personnaliser le fonctionnement du script en utilisant des balises meta dans le code du fichier HTML (via le modèle HTML de HA, dans la partie "HEAD").
Les options proposées par André n'ont pas été conservées, puisqu'on peut obtenir le même résultat en choisissant correctement les options d'export du fichier.
Des explications plus détaillées sont données dans un fichier PDF situé dans le ZIP fourni en attachement.
Pour faire rapidement, voici les principales modifications apportées via les balises :
myr-tempo = yes/no => permet de conserver le métronome sans le changement de tempo
myr-offset = nombre => permet de décaler en hauteur (+ ou -) la zone de détection des mesures
myr-color = valeur =>couleur nommée ou code HTML pour les indications de lecture
myr-playChar = valeur =>code HTML ou caractère à afficher pour jouer une mesure
myr-repeatChar = valeur => code HTML ou caractère à afficher pour répéter une mesure
myr-opacity = Nombre =>valeur (0-100) du pourcentage d'opacité à appliquer aux indications
myr-noSoloLevel = Nombre => valeur (0-100) du pourcentage de niveau audio à laisser aux autres pistes (activation d'une piste en solo dans le mixer, muet par défaut)
myr-size = nombre => permet de forcer la taille des n° de mesures ou des indications (bug si le nombre de portées est différent entre les systèmes)
Les icônes d'impression, de transposition (fonction 'detune'), de téléchargement peuvent être inhibées depuis HA comme expliqué dans le document PDF.
Les fonctions ajoutées sont : getMyrMeta / isDisabled / alertZoom / isInMeasure / getPosition
Les fonctions modifiées sont : MyrInitAllAtOnLoad / MyrManageMouseOnObj / MyrCreateBarNum /MyrAddMenuItemsTempo / MyrSoundUpdateTrackLevels / MyrSoundDepause
J'ai l'habitude des scripts (VB, VBA, Bash, PHP, etc.) mais pas trop du Javascript, il est sûrement possible de faire mieux mais pour le moment c'est fonctionnel et ça semble fiable d'après mes tests.
En cas de problème, remettre le script d'origine et me signaler le soucis, j'essaierai de le corriger.
Bons tests, et merci de lire le document PDF joint avant d'utiliser le script.
Jacques
|
|
 offline |
Linux Mageia release 8 (Official) for x86_64 (et éventuellement Windows 7)
Harmony Assistant 9.9.6c.0 x64
PDFToMusic 1.7.5c x64
|
|
|
JP
Board Master
WWW | Email
Gender: 
Posts: 3643
|
 |
Re: Nouvelle version du script myrweb.js avec Play/Repeat
« Reply #1 on: May 2nd, 2023, 3:31pm » |
Quote | Modify
|
Bravo pour ce délicat travail.
Quelques remarques d'un utilisateur quotidien de JavaScript, mais ce n'est peut-être pas important :
La syntaxe de javascript a évolué ces dernières années conformément aux spécifications EcmaScript ES6 et suivantes.
Tel qu'il est écrit (notamment avec des "let" et "const") ce script ne peut être utilisé qu'avec des versions de navigateurs internet supportant ES6+, ce qui me semble acceptable pour la plupart des utilisateurs de Myrweb.
Mais je constate dans les notes du Forum et dans le Blog que certains utilisateurs ont des demandes concernant de très anciennes versions des systèmes d'exploitation. La question est "est-ce que ces utilisateurs utilisent également de très anciennes versions des navigateurs ?"
- si la réponse est "oui", il faudrait supprimer de ton code les "let" et "const" et les remplacer par des "var". (Je n'ai pas trouvé dans ce code d'autres fonctionnalités ES6+ mais il y en a peut-être)
- si la réponse est "non", on pourrait simplifier le code en utilisant la syntaxe ES6+
Par exemple la première fonction
Code:
function getMyrMeta(name,def) {
metas = document.getElementsByTagName('meta');
if (typeof def === null) def = '';
var val = def;
for (let i = 0; i < metas.length; i++) {
if (metas[i].getAttribute('name') === 'myr-' + name) {
val = metas[i].getAttribute('content');
}
}
console.log('getMyrMeta(' + name + ',' + def + ') => ' + val);
return val;
}
|
|
Pourrait s'écrire
Code:
function getMyrMeta(name, def = '') {
const metas = document.getElementsByTagName('meta');
for(const meta of metas) {
if(meta.getAttribute('name') == 'myr-' + name)
return meta.getAttribute('content');
}
return def;
}
|
|
L'utilisation de la définition des valeurs par défaut des arguments de la fonction simplifie le code, de même la syntaxe for(...of...) qui n'existait pas dans les versions précédentes.
Je vois aussi que tu as laissé des commandes "console.log(...);"
qui sont indispensable pendant les tests mais qu'il serait bien de mettre en commentaires avant diffusion générale du fichier script.
|
|
offline |
|
|
|
ANdre_B
Board Master
Email
Gender:
Posts: 3522
|
|
Re: Nouvelle version du script myrweb.js avec Play/Repeat
« Reply #2 on: May 2nd, 2023, 4:27pm » |
Quote | Modify
|
Travail très prometteur, bravo.
Mes propres modifications étaient vraiment du "quick and dirty", mais il y a eu un temps où d'autres modifs - de quelqu'un d'autre dont j'ai oublié le nom - se combinaient à tout ceci, mais supprimées ensuite dans mes éditions du .js
Les solutions proposées me semblent de bien meilleures alternatives.
Je viens de lire la documentation, il y a certes quelques corrections que je pense suggérer, mais je vais d'abord examiner et essayer tout cela en détail. Pour la clarté, j'aurais aimé voir un exemple de myrweb_template incluant les balises au autres adaptations.
|
|
offline |
André Baeck, de retour en Belgique après 12 ans passés dans le Gard.
Windows 11, HA 2026h1e (et précédents)
|
|
|
JacquesF
Board Newbie
Posts: 25
|
|
Re: Nouvelle version du script myrweb.js avec Play/Repeat
« Reply #3 on: May 2nd, 2023, 9:41pm » |
Quote | Modify
|
Merci pour les retours.
Pour les let et const, je vais par prudence revenir à la déclaration var.
Je ne déclare pas de constantes, celles présentes dans le script sont celles d'origine et je ne vais pas modifier le reste du code. Ce code est plutôt complexe et comme il a été minifié, le nom des variables n'aide pas à sa compréhension. Et tant pis pour les évolutions ECMA, je pense que par soucis d'homogénéité le format des boucles doit rester le même que celui des concepteurs.
Et compte tenu des modifications indiquées plus bas, je vais changer le nom de la balise pour le niveau de mute des autres pistes de noSoloLevel en muteLevel.
Je vais rajouter un modèle HTML dans la prochaine livraison.
Mais en voici déjà un Code:
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
<head>
<meta charset="UTF-8" />
<meta name="myr-tempo" content="no" />
<meta name="myr-color" content="#C9A40E" />
<meta name="myr-size" content="60" />
<meta name="myr-muteLevel" content="25" />
<meta name="myr-muteName" content="Voix muettes" />
<script src="./myrweb.js"></script>
<title>$TITRE$</title>
</head>
<body style="font-family:sans-serif;font-size:12px;background-color:#E0A070">
<center><h1>$TITRE$</h1></center>
<noscript>
<center><h1 style="color:#902020;font-style: italic;">MyrWeb cannot be used because Javascript is turned off.</h1></center>
</noscript>
<center>
<div class="myrweb" style="width:700px; height:200px;" data-myrweb="" data-id="$ID$">
<script>
var data_$ID$_varb64="$DATA$"
</script>
</div>
</center>
</body>
</html> |
|
Je suis en train de fignoler un peu le réglage du niveau audio des pistes 'mutées'. Le fait de passer par une balise meta permettait d'avoir une valeur modifiable par la personne qui créé le fichier myrweb sans avoir à mettre les doigts dans le code, mais pour la changer localement ça devenait lourd.
Finalement, j'ai trouvé une solution en rajoutant une portée vide dans la partition dont le nom devra être spécifiée dans le fichier HTML. Dans ce cas, il faut masquer les portées vides à l'impression pour que ce soit correct, mais il est peut-être possible de trouver d'autres astuces (réduire sa taille au minimum, avec pourquoi pas 1 ou 0 lignes et en blanc sur fond blanc) pour la masquer si on a besoin d'afficher les portées vides. Je fouillerai un peu plus dans cette direction.
En indiquant le nom, le script va rechercher à quelle piste audio elle correspond et lui affecter à l'init la valeur 0 ou celle indiqué via la balise meta.
Ensuite, ouvrir le mixer présentera cette piste avec un nom éloquent (Voix muettes par exemple) et on pourra régler le niveau sonore. Le fait d'activer une piste en solo forcera le niveau des autres voix à cette valeur.
Le choix du nom de cette piste étant libre, cela fonctionnera quelque soit la langue de création du fichier si son concepteur prend le soin de la spécifier correctement.
En l'absence de valeurs concernant le mute, les niveaux seront forcés à 0.
Si le nom ou la piste spécifique manque, le niveau sera forcé selon la valeur de la base meta mais le réglage sera impossible par l'utilisateur.
Bonne soirée
Jacques
|
|
offline |
Linux Mageia release 8 (Official) for x86_64 (et éventuellement Windows 7)
Harmony Assistant 9.9.6c.0 x64
PDFToMusic 1.7.5c x64
|
|
|
JacquesF
Board Newbie
Posts: 25
|
Voici la version 1.1 du script.
Comme indiqué plus haut, celui-ci permet maintenant de régler directement via l'interface du mixeur le niveau des pistes qui sont assourdies en mode "solo".
Pour que cela fonctionne, il faut créer une portée vide dans la partition et masquer l'impression des portées vides dans la mise en page.
Si l'on doit toutefois les afficher, il est possible d'utiliser une piste numérique à la place et de configurer ses paramètres pour masquer l'affichage. En réduisant au maximum sa hauteur, elle ne modifie pas la mise en page. Cette technique est décrite dans le fichier PDF joint dans le ZIP.
Le nom de cette piste doit être suffisamment court et explicite pour l'utilisateur et en tant que portée dans HA, il est préférable de la positionner soit tout en haut, soit tout en bas de la liste des portées pour que le résultat dans le mixeur soit clair. Il faut l'indiquer dans le fichier HTML à l'aide la balise meta myr-muteName.
Si la partition HA est exportée et que le nom n'est pas déclaré dans le fichier HTML, alors cette piste sera affichée dans le mixeur avec le niveau par défaut (réglage dans le mode avancé) à 100%. Le fait de le changer ne modifiera rien au niveau utilisateur. Si le niveau sonore n'est pas indiqué dans le fichier HTML, alors la piste sera initialisée à 0 et le fonctionnement sera identique au script d'origine.
Si le nom est déclaré dans le fichier HTML et que la partition ne contient pas cette piste, rien ne sera affiché dans le mixeur et le niveau des pistes assourdies prendra la valeur définie dans la balise myr-muteLevel si elle est présente, sinon on revient au fonctionnement d'origine avec la valeur à 0.
Pour faire suite aux remarques de JP, la déclaration des variables est faite par le mot clef "var" pour rester compatible. Toutefois, je n'ai pas modifié le code d'origine volontairement.
Les traces dans la console sont volontaires, il y en a d'ailleurs beaucoup dans le code d'origine. Comme personne ne regarde cette console, elles ne gênent pas à l'utilisation et permettent de savoir si les balises meta sont bien prises en compte (syntaxe et valeurs correctes) sans avoir à modifier le code du script (surtout si on utilise la version "minifiée").
Un exemple de modèle HTML avec quelques balises est joint au fichier ZIP.
Le fonctionnement n'est probablement pas optimisé puisque la règle prise lors de l'écriture était de modifier au minimum le code d'origine. De plus comme ce code a été minifié, le nom des variables ne signifie plus rien ce qui ne facilite pas la compréhension du fonctionnement.
Pour configurer le niveau sonore de la piste qui sert à régler les voix assourdies, j'ai utilisé la fonction MyrMixDoItem qui est appelée dès l'ouverture du mixeur. Mais comme elle est aussi appelé à chaque changement des valeurs du mixeur, on rappelle la fonction d'init des valeurs liées au mode solo et et les réécrit plusieurs fois de suite. C'est sans conséquence mais pas très optimisé.
Petit bug qui reste, la sélection des mesures à jouer ou répéter dans les vues n'est pas toujours très précise. Mais on peut sélectionner dans la vue générale puis passer dans une autre vue en cas de problème.
Jacques
|
| « Last Edit: May 3rd, 2023, 12:59am by JacquesF » |
offline |
Linux Mageia release 8 (Official) for x86_64 (et éventuellement Windows 7)
Harmony Assistant 9.9.6c.0 x64
PDFToMusic 1.7.5c x64
|
|
|
ANdre_B
Board Master
Email
Gender:
Posts: 3522
|
|
Re: Nouvelle version du script myrweb.js avec Play/Repeat
« Reply #5 on: May 3rd, 2023, 8:25am » |
Quote | Modify
|
Merci Jacques, pour les exemples de balises meta. Je n'y étais pas arrivé hier soir.
Ce que je suggérerais, c'est de refléter les références de la version, à la ligne:
var myrweb_version = "0.3.5 Build 1 Compiled Mon Jan 10 09:03:51 2022",
Je n'y suis arrivé qu'en respectant scrupuleusement la longueur de la variable.
De cette façon, on verra cette référence en cliquant l'icône "myrweb" à l'extrême droite de la ligne-menu.
|
|
offline |
André Baeck, de retour en Belgique après 12 ans passés dans le Gard.
Windows 11, HA 2026h1e (et précédents)
|
|
|
JP
Board Master
WWW | Email
Gender:
Posts: 3643
|
|
Re: Nouvelle version du script myrweb.js avec Play/Repeat
« Reply #6 on: May 3rd, 2023, 10:09am » |
Quote | Modify
|
Du point de vue JavaScript je suis d’accord avec le fait de rester compatible avec les anciennes versions du langage.
Je gère actuellement 2 sites, j’en ai un dont le connais les utilisateurs que j’ai passé en ES6 et un autre qui a comme utilisateurs la communauté mondiale des égyptologues et que j’ai laissé dans l’ancienne version par précaution.
Du point de vue Myrweb, je ne pense pas utiliser cette version pour deux raisons :
1) je crains les conséquences de l’évolution des versions de HA. Utiliser le script officiel garantit plus ou moins qu’il évoluera en fonction des évolutions de HA
2) ma communauté d’utilisateurs est habituée à l’ergonomie actuelle de Myrweb et toute modification de cette ergonomie serait plus dommageable qu’utile
La seule fonction qu’il me semblerait utile d’ajouter est la sélection des mesures à jouer et à faire tourner en boucle.
|
|
offline |
|
|
|
JacquesF
Board Newbie
Posts: 25
|
|
Re: Nouvelle version du script myrweb.js avec Play/Repeat
« Reply #7 on: May 3rd, 2023, 6:27pm » |
Quote | Modify
|
Bonjour JP,
Si tu as besoin des fonctions lire à partir de et répéter, alors utilise le script sans rien personnaliser. Le fonctionnement est le même que le script d'origine, à part ces 2 fonctions.
Tout ce qui est personnalisation comme le niveau sonore pour les autres pistes en solo reste aux valeurs d'avant si rien n'est indiqué dans l'entête HTML.
Pour André, je vais voir pour indiquer éventuellement la version du code des modifications ajoutées. Personnellement, je désactive le bouton Myrweb, étant donné que je suis le seul à gérer le site de la chorale, je connais la version, et pour ce qui est des choristes c'est le dernier de leurs soucis.
Je fournirai d'ici peu encore une évolution ou deux, la première (ça marche déjà) est un bouton d'aide qui ouvre une page Web en popup. La seconde, si j'y arrive, est de permettre de définir une mesure de fin qui ne soit pas la dernière sans impliquer de passer en mode répétition. Dans la version d'origine, et donc celle-ci aussi, le 1er clic démarre la sélection de la mesure jusqu'à la fin. Si on clique sur une mesure avant, elle modifie le point de départ sans toucher à la fin. Si on clique après la 1ère mesure, alors on passe en répétition de la 1ere à celle où l'on vient de cliquer. Re-cliquer sur une mesure avant modifie juste la plage de répétition.
L'idée serait de ne basculer en mode répétition que si la mesure de fin est identique à celle déjà sélectionnée, ce qui permettrait d'ajouter une plage de jeu sans aller à la fin.
À suivre donc...
Jacques
|
|
offline |
Linux Mageia release 8 (Official) for x86_64 (et éventuellement Windows 7)
Harmony Assistant 9.9.6c.0 x64
PDFToMusic 1.7.5c x64
|
|
|
ANdre_B
Board Master
Email
Gender:
Posts: 3522
|
|
Re: Nouvelle version du script myrweb.js avec Play/Repeat
« Reply #8 on: May 4th, 2023, 5:45am » |
Quote | Modify
|
on May 3rd, 2023, 6:27pm, JacquesF wrote:| Pour André, je vais voir pour indiquer éventuellement la version du code des modifications ajoutées. Personnellement, je désactive le bouton Myrweb, étant donné que je suis le seul à gérer le site de la chorale, je connais la version, et pour ce qui est des choristes c'est le dernier de leurs soucis. |
|
Certainement, ce n'est pas à l'intention des choristes. Mais pour le développeur, ou l'administrateur de site, qui doit parfois jongler avec différentes versions, il est difficile de se souvenir avec laquelle un myrweb.html donné a été généré.
Enfin, c'est toi qui vois.... (comme on dit dans le Nord et en Belgique).
|
|
offline |
André Baeck, de retour en Belgique après 12 ans passés dans le Gard.
Windows 11, HA 2026h1e (et précédents)
|
|
|
ANdre_B
Board Master
Email
Gender:
Posts: 3522
|
|
Re: Nouvelle version du script myrweb.js avec Play/Repeat
« Reply #9 on: May 4th, 2023, 5:50am » |
Quote | Modify
|
J'utilise myrweb surtout à partir de MyrScript, car je fais de la segmentation et j'en crée un pour chaque portée de chaque segment.
Je vais examiner comment gérer les balises meta dans ce cas.
|
| « Last Edit: May 4th, 2023, 5:56am by ANdre_B » |
offline |
André Baeck, de retour en Belgique après 12 ans passés dans le Gard.
Windows 11, HA 2026h1e (et précédents)
|
|
|
ANdre_B
Board Master
Email
Gender:
Posts: 3522
|
|
Re: Nouvelle version du script myrweb.js avec Play/Repeat
« Reply #10 on: May 4th, 2023, 5:56am » |
Quote | Modify
|
Le bouton de téléchargement est en effet dépendant des options d'inclusion des "pièces jointes".
Mais le bouton d'impression n'est pas contrôlé, si j'ai bien compris, et une balise meta serait bienvenue là aussi.
J'ai vu des situations de réticences à Myrweb pour des raisons de droit de copie de la partition, d'autant plus si l'utilisateur peut créer sa propre copie plutôt que d'acheter sa partition (cas d'oeuvres qui ne sont pas dans le domaine public, ou de discipline/uniformité voulue par le chef de choeur).
|
|
offline |
André Baeck, de retour en Belgique après 12 ans passés dans le Gard.
Windows 11, HA 2026h1e (et précédents)
|
|
|
JacquesF
Board Newbie
Posts: 25
|
on May 4th, 2023, 5:56am, ANdre_B wrote:Le bouton de téléchargement est en effet dépendant des options d'inclusion des "pièces jointes".
Mais le bouton d'impression n'est pas contrôlé, si j'ai bien compris, et une balise meta serait bienvenue là aussi. |
|
André, comme je l'ai indiqué dans le PDF, le bouton d'impression ne pose pas de problème. Dans la configuration avancée, au moment de l'export (ou dans le fichier MyrWebHarmony.setup), il suffit de mettre l'option "-print" dans les paramètres du toolset pour le désactiver. Si tu veux une balise meta en plus pour le faire, dis le moi et ça ne posera pas de problème. Mais je suis parti du principe de ne modifier le code que quand la configuration ne permettait pas d'obtenir le même résultat.
Dans la version 1.2 du code, pas encore livrée, mes paramètres dans le fichier MyrWebHarmony.setup sont :
commands "titlebar=no;theme-fill=#C9A40E;toolset=default -detune -print -info help;"
-detune masque bien l'icone du diapason, -print celle de l'impression, -info le bouton Myriad et help rajoute une bouton avec un "?" pour ouvrir une page web en popup (qui indique maintenant la version de l'addon) comme dans les images jointes.
Pour info, la version est affichée dans la fenêtre en décomposant la chaîne de caractères dans un tableau, et le code ne va pas au-delà de la limite prévue.
Pour afficher la version de mon code, j'ai modifié le code qui créé la fenêtre du menu.
Jacques
|
| « Last Edit: May 4th, 2023, 8:23am by JacquesF » |
offline |
Linux Mageia release 8 (Official) for x86_64 (et éventuellement Windows 7)
Harmony Assistant 9.9.6c.0 x64
PDFToMusic 1.7.5c x64
|
|
|
JacquesF
Board Newbie
Posts: 25
|
|
Re: Nouvelle version du script myrweb.js avec Play/Repeat  ha-barre.png
« Reply #12 on: May 4th, 2023, 8:23am » |
Quote | Modify
|
On ne peut pas attacher 2 images à la fois, voici donc la barre d'outils avec l'option '-print'
|
| « Last Edit: May 4th, 2023, 8:24am by JacquesF » |
offline |
Linux Mageia release 8 (Official) for x86_64 (et éventuellement Windows 7)
Harmony Assistant 9.9.6c.0 x64
PDFToMusic 1.7.5c x64
|
|
|
JP
Board Master
WWW | Email
Gender:
Posts: 3643
|
|
Re: Nouvelle version du script myrweb.js avec Play/Repeat
« Reply #13 on: May 4th, 2023, 9:55am » |
Quote | Modify
|
on May 4th, 2023, 5:56am, ANdre_B wrote:| J'ai vu des situations de réticences à Myrweb pour des raisons de droit de copie de la partition, d'autant plus si l'utilisateur peut créer sa propre copie plutôt que d'acheter sa partition (cas d’½uvres qui ne sont pas dans le domaine public, ou de discipline/uniformité voulue par le chef de ch½ur). |
|
En ce qui me concerne, je fais des fichiers Myrweb avec une mise en page difficilement imprimable: 1 système par page, 32 cm de largeur, 22 cm de hauteur. Cela n'incite pas à imprimer ça sur du papier.
|
|
offline |
|
|
|
ANdre_B
Board Master
Email
Gender:
Posts: 3522
|
|
Re: Nouvelle version du script myrweb.js avec Play/Repeat
« Reply #14 on: May 4th, 2023, 10:53am » |
Quote | Modify
|
on May 4th, 2023, 9:55am, JP wrote:
En ce qui me concerne, je fais des fichiers Myrweb avec une mise en page difficilement imprimable: 1 système par page, 32 cm de largeur, 22 cm de hauteur. Cela n'incite pas à imprimer ça sur du papier. |
|
Encore qu'avec une légère réduction d'échelle, en paysage et/ou deux pages par feuille, ça puisse le faire.
Mais aussi, c'est parfois une question de principe: j'ai eu à disposition des sources .myr avec autorisation d'exporter mais pas d'imprimer.
|
|
offline |
André Baeck, de retour en Belgique après 12 ans passés dans le Gard.
Windows 11, HA 2026h1e (et précédents)
|
|
|
|
