@KillianKemps il y a des trucs que je comprends pas dans le code de l'extension Qowala. Genre dans le background script tu s une fonction showUrls qui transforme des objets représentants des onglets en un objet qui associe leurs URLs à true. Mais ensuite tu ne fait que utiliser les URLs (que tu récupère avec Object.keys).
Pourquoi pas simplement faire:
tabs.map(tab => new URL(tab.url))
?
@Bat Oui tu as raison il y a des parties du code qui peuvent paraître bizarre car j'expérimentais mais je n'ai pas factorisé ensuite.
Dans le cas du showUrl tu peux en effet faire un simple map et il faut prendre en compte le cas du hostname vide (quand l'utilisateur est sur une page about:config par exemple).
N'hésite pas à factoriser ces parties qui te gênent et à améliorer la qualité du code, je t'avoue que je me suis dépêché de coder le plugin 😀
@KillianKemps Okay je vais en profiter pour nettoyer un peu tout ce que je trouve sale. 😁
@Bat Haha merci !
Les designers vont améliorer le visuel que je t'ai déjà passé sur Github. Est-ce que tu peux me donner ton adresse email par DM ou comme tu veux, pour qu'on t'invite dans le service où on fait les maquettes et que tu puisses voir les visuels ? (le service c'est figma.com qui permet de faire des maquettes de manière collaborative)
@Bat Merci, on t'a ajouté au Figma ;)
@Bat Dans un premier temps on propose de faire plus simple que ce que je t'ai proposé au départ sur Github.
Au lieu de créer une page spécialement pour les paramètres, on va juste changer la popup pour qu'elle propose de tracker le site web courant au lieu du "This website is not tracked!" qu'elle affiche actuellement. Donc tu ne devrais pas avoir grand chose à faire en HTML/CSS.
@French_Villain va te faire un visuel d'ici peu ;)
(J'espère que je ne te dis pas ça trop tard ^^')
@KillianKemps @French_Villain Euh... ben un peu si. 😂 Mais c'est pas grave, je ferait ça en plus aussi, dès que j'ai la maquette.
@Bat @French_Villain Arf, désolé j'essayerais de te donner les bonnes infos plus tôt la prochaine fois. ^^'. On va préparer les prochaines maquettes de sorte qu'elles soient prêtes si tu veux continuer à contribuer sur le projet Qowala.
Si tu veux tu peux aussi ouvrir une PR sur Github avec un label "WIP", comme ça on pourra se synchroniser plus facilement.
@KillianKemps @French_Villain Okay, je règle deux trois petits bugs avec ce que j'ai fait et j'ouvre un PR.
@KillianKemps @French_Villain Tadaaaa ! 🎉 Je vais ouvrir une PR !
@French_Villain @KillianKemps Bon voilà le screen, je sais pas pourquoi il est pas parti...
@Bat @French_Villain Super ! Ça va faire plaisir aux mastonautes.
Je vois que le path `/about` de l'URL est aussi affiché dans la popup.
Tu as pu voir s'il comptabilisait le temps aussi si tu es sur une autre page que le `/about` ?
Et est-ce que c'est possible de faire en sorte qu'il n'y ait que le nom de domaine affiché ?
(Juste `unixcorn.xyz` au lieu de `https://unixcorn.xyz/about`)
Ça permettra d'avoir une pop-up moins large.
@KillianKemps @French_Villain C'est fixé normalement. Je suis pas complètement satisfait du code dans popup.js, mais je pense que tu peux merger, je m'en occuperait plus tard au pire. 🙂
@Bat @French_Villain J'ai vu ton boulot dans la PR Github, c'est génial ! Je me sens honteux de t'avoir passé un code dans cet état mais tu l'as bien amélioré.
Je t'ai juste mis une demande de changement car je pense qu'une méthode que tu utilises casse la compatibilité avec Chrome.
En tout cas merci !
@KillianKemps @French_Villain Ça devrait être bon !
@Bat @French_Villain Cool ! Je vais merger, tu veux pas ajouter ton nom à la liste des contributeurs avant ?
@Bat @French_Villain C'est mergé ! Merci pour ta PR.
Si tu veux, on te tient au courant quand la maquette sera terminée pour adapter la popup.
Si t'es intéressé pour contribuer plus régulièrement, tu peux assister à notre prochaine réunion hebdomadaire et on pourra t'expliquer ce qu'on a prévu pour le plugin Qowala.
(Ah j'ai découvert un léger bug quand on entre juste un domaine, du coup l'autovalidation HTML attend une URL complète haha :D)
@KillianKemps @French_Villain Yep, ça m'intéresse d'assister à la réunion. Vous utilisez quoi pour faire ça ?
Et pour le bug, c'est-à-dire ? Genre "unixcorn.xyz" ça marche pas ? J'avais enlevé l'autovalidation HTML des URLs pour en mettre une perso, qui rends les http(s):// optionnel.
@Bat @French_Villain Cool, on fait ça le dimanche soir vers 18h30, ça te va ?
On utilise meet.jit.si pour faire la conf', donc pas besoin de logiciel, juste un navigateur récent.
Pour le bug, un nom de domaine tout court ne marche pas. J'ai une erreur dans la console : "TypeError: getgrav.org is not a valid URL." quand je n'entre que "getgrav.org". L'erreur est à la ligne 19 du fichier options.js
@KillianKemps @French_Villain OK, ça devrait être bon pour assister à la réunion.
Je vais regarder pour le bug. Tu as quel navigateur (et quelle version) ?
@Bat @French_Villain Super, on t'enverras le lien Jitsi pour nous rejoindre dimanche.
J'ai testé dans Firefox Developer Edition version 52.0a2
@KillianKemps C'est baptiste@gelez.xyz 🙂