JS refactoring

[TiTi]

Les gros travaux de refactoring sont terminés. Le code est désormais mieux structuré. Tout marche comme avant (au bug près). A l'exception des customQueries qu'on ne peux pas envoyer, mais c'est juste une histoire de branchement. Sera effectué bientôt.

Les modifs ne concernent que ce qui est sous /html/js/ (partie cliente). [sauf .gitignore, des fix d'orthographe et un fix de crash serveur]

[trasher4boss]

Avant de merger le pull request j'ai quelques remarques :

• Le player flash fait un auto connect. Ce qui rajoute forcément un client à la radio. Personnellement je n'utilise pas le player flash (comme lilian et camille) et je me sers de l'interface web que pour contrôler la radio. C'est donc pas le comportement désiré pour nous. De plus le stop stream c'est un peu du fake vu que le client reste connecté et, est donc toujours connecté à la socket ce qui mange de la bande passante pour rien.

Et en moins génant pour le merge :

• On ne peut plus faire une recherche vide pour avoir toute la libraire. (la recherche avec % fonctionne)
• Sur les résultats de recherche les numéros de pages sont collés ( changement de style ).

Sinon tu nous a fait de belles features avec le player flash, et toute l'interface est plus harmonieuse ;) les styles passent bien maintenant sous mon firefox 12 sous BSD \o/

[TiTi]

• Yes, y'a un changement de comportement pour le flash. Le but était de montrer comment instancier le jukebox dans index.html Je vais enlever ca dans un prochain commit.
• Oui j'avais vu que ca continue à bouffer de la bande passante malgré .stop(), c'est pour ca que j'ai rajouté la méthode .stopStreaming() :-D

Ok je vais fixer ca aussi :

• Recherche vide
• Numéros de pages collés

Oui y'a 2-3 trucs nouveau / changé dans les styles, genre :

• Slider pour le volume
• Tableau de recherche à 100% width
• Slider de pagination à 100% width

Je rajouterais ces commits d'ici ce soir, donc attends avant de merger stp.

[trasher4boss]

Ça roule. Je vais jeter un oeil aussi sur le lancement du flash. Parce que j'ai pas capté si le stop streaming était utilisé dans l'interface. Sinon les pb de style c'est vraiment pas grave. Ça se corrige rapidement, comme pour les recherches. Dès que j'arrive au taff je demande aux autres ce qu'ils préfèrent comme comportement pour le flash. Attend aussi un peu avant de proposer test commits alors. On sait jamais si ils se mettent a préférer un mode full web par défaut. Le 23 oct. 2012 09:16, "TiTi" notifications@github.com a écrit :

• Yes, y'a un changement de comportement pour le flash. Le but était de montrer comment instancier le jukebox dans index.html Je vais enlever ca dans un prochain commit.
• Oui j'avais vu que ca continue à bouffer de la bande passante malgré .stop(), c'est pour ca que j'ai rajouté la méthode .stopStreaming() :-D

Ok je vais fixer ca aussi :

• Recherche vide
• Numéros de pages collés

Oui y'a 2-3 trucs nouveau / changé dans les styles, genre :

• Slider pour le volume
• Tableau de recherche à 100% width
• Slider de pagination à 100% width

Je rajouterais ces commits d'ici ce soir, donc attends avant de merger stp.

— Reply to this email directly or view it on GitHubhttps://github.com/nestorInc/jukebox/pull/6#issuecomment-9692625.

[TiTi]

Bon tout ca est désormais fixé, mais ca n'apparaît pas dans cette request alors je vais la fermer et en réouvrir une autre. A noter que j'ai fais comme ceci pour l'autoconnect :

• Par défaut index.html est en autoconnect
• Si "#noPlay" est présent dans l'url (sensible à la casse), alors l'autoconnect est désactivé : pas de streaming (comme avant)
• Si l'utilisateur fait stop via la GUI, ou la console : on continue de streamer 30s au cas ou il refasse start/play
• Si les 30s sont atteinte après un stop : le streaming est stoppé automatiquement
• Réappuyer sur "PlayStream" marche (ou .play() dans la console)