Design et ergonomie incohérent

ghost commented 8 years ago

Le design est vraiment incohérent et plutot très très moche. Bref entre la connexion sur la page admin (noir, gris, orange, rouge) et une fois identifié le design change (gris, noir, blanc, bleu, vert, rouge) youhou.... magggnniiifffiique !!!!! Rajoutez encore des couleurs le violet, le fushia, le jaune, le turquoise etc... ça devrait être mieux OOOooo Et l'image est sortie d'un vieux film ultra SD des années 30 ou l'avez-vous pompé à une secte (rael, la scientologie) ?

Pourquoi devoir aller sur une page admin.php par ailleurs ? Par sécurité ? Omg....

Sinon je me demandais si la webapp était mon amie pour me tutoyer, purer ça fait vraiment amateur comme travail !

Faire un glisser - déposer sur une version mobile c'est vraiment très con !! Le responsive design permettrait pourtant d'ajouter une bonne méthode à l'ancienne dans ce cas....

Le responsive design n'est pas adapté ici pour les écrans 800*480 à cause du bloc glisser - déposer qui est trop grand...

Pourquoi ajouter un lien vers un blog ? Le votre surement, encore le github ça se comprend mais pour une web app ajouter un lien vers son blog c'est tellement has been....

J'aurais surement d'autres critique à formuler (mais j'ai pas encore regardé le code...), mais là à forker, c'est trop long pour le peu d'utiliter que j'en aurais, néanmoins l'idée est bonne, si j'avais un peu de temps je réfléchirai sur un design plus sobre et épuré à votre place.

Bonne continuation en tout cas. (Je suis sévère, normal, c'est pas en prenant des pincettes que qq chose avance, cyrille en sait qq chose)

broncowdd commented 8 years ago

C'est clair, l'affichage via light box est bien mieux et si on peut afficher directement davantages de formats, c'est un plus. L'affichage pdf etc n'est pas difficile à gérer en soi (une variable a changer)... L'idée d'un éditeur de texte minimaliste nous a également effleuré pour tout dire, t'as qu'à voir comme c'est une excellente idée ;-) J'en aurai l'usage. Stéphane m'a donné une bonne idée: cacher la zone d'upload avec un bouton Upload pour la révéler... C'est sympa car ça fait moins chargé. Du coup, j'ai poussé le concept plus loin: en cliquant sur upload, on déploie la zone de d&d en fullscreen... Ben ça me semble vraiment mieux.

Je vais essayer d'intégrer ta structure à ma version 2.3 de la façon la plus propre et maintenable possible dans la semaine et je te la passe, ok?

Le 17 février 2016 12:21:53 CET, EauLand notifications@github.com a écrit :

Ah ok je me demandais qd était utilisé la galerie que j'ai vu dans les css et le php. Je vais regarder ça car bon j'avais enlevé les css ne trouvant pas cette option.

Sinon bien que ce post ne soit pas approprié pour ça je le met quand même ici (oui j'ai pas encore cherché) quand je télécharge un fichier texte, bozon me retourne la page html qui l'affiche avec la visu avec l'extension txt, arf.

J'aimerai aussi comme pour la lightbox que les fichiers pdf, odt, doc et même txt s'affiche dans une fenêtre sans changer de page et qu'il soit possible d'éditer les fichiers textes au minimum. Pas mal de js permet de faire ça donc je continue sur l'intégration, pour améliorer l'interface du mieux que je peux, en espérant que ça plaira.

Reply to this email directly or view it on GitHub: https://github.com/broncowdd/BoZoN/issues/57#issuecomment-185159706

Envoyé de mon appareil Android avec K-9 Mail. Veuillez excuser ma brièveté.

Draky50110 commented 8 years ago

@broncowdd Au vu de tout ce qui va arriver en 2.3, tu devrais aussi changer de nom.

Un truc qui rappellerai ce que peux faire ton petit script. Un truc genre "ownCloud by Bronco" :D

ghost commented 8 years ago

Pas de problème bronco, ça me va. Je regarderai tes modifs.

Je m'occupe des playlists m3u déjà, le reste je regarde plus tard, il faut déjà que je trouve où l'extension est mise comme ok, en modifiant le mime-type je croyais que ça allait marcher.

ghost commented 8 years ago

@Draky50110 ownMate ptdr :)

broncowdd commented 8 years ago

^^

Je me demandais également si, pour la version desktop, on ne pourrait pas faire un menu contextuel à la place des icônes... (et du coup, trouver une alternative pour mobile)

La seule chose qui m'empêche de supprimer les trois modes (view/links/move) c'est l'accumulation d'icônes... Mon problème reste que je veux aussi limiter le nb de clics pour atteindre une fonction...

ghost commented 8 years ago

C'est vrai que c'est pas super ergonomique, mais j'avais conservé la logique précédente un peu.

Faire un menu contextuel sur les fichiers (en mode "..." like dropbox) c'est faisable mais ça va demander pas mal de petits changements, ou pas.

Je finis l'intégration des playlists audio m3u et je regarde ça. La variable c'était behavior à renseigner :)

broncowdd commented 8 years ago

Le changement, c'est maintenant ;-) Sinon, je pense que ce serait possible sans bouleversements. Il y aurait peut être même de la simplification à la clé...

Le 17 février 2016 14:42:57 CET, EauLand notifications@github.com a écrit :

C'est vrai que c'est pas super ergonomique, mais j'avais conservé la logique précédente un peu.

Faire un menu contextuel sur les fichiers (en mode "..." like dropbox) c'est faisable mais ça va demander pas mal de petits changements, ou pas.

Je finis l'intégration des playlists audio m3u et je regarde ça. La variable c'était behavior à renseigner :)

Reply to this email directly or view it on GitHub: https://github.com/broncowdd/BoZoN/issues/57#issuecomment-185208947

Envoyé de mon appareil Android avec K-9 Mail. Veuillez excuser ma brièveté.

broncowdd commented 8 years ago

Au passage, je viens de voir que tu utilises jquery: je préfère éviter et rester sur du js vanilla.

Le 17 février 2016 14:42:57 CET, EauLand notifications@github.com a écrit :

C'est vrai que c'est pas super ergonomique, mais j'avais conservé la logique précédente un peu.

Faire un menu contextuel sur les fichiers (en mode "..." like dropbox) c'est faisable mais ça va demander pas mal de petits changements, ou pas.

Je finis l'intégration des playlists audio m3u et je regarde ça. La variable c'était behavior à renseigner :)

Reply to this email directly or view it on GitHub: https://github.com/broncowdd/BoZoN/issues/57#issuecomment-185208947

Envoyé de mon appareil Android avec K-9 Mail. Veuillez excuser ma brièveté.

ghost commented 8 years ago

Ok je vais utiliser autre chose, par contre pourrais tu regarder pourquoi le readfile modifie mes fichiers (m3u ici...)... j'ai vu ça dans le fichier commands_GET_vars readfile(f)...

Là je peux pas avancer, puisque mon readfile enlève mon extension.

Exemple: http://bozon.unelio.xyz/index.php?f=156c45c9e710ad donne: http://radiofg.imp

au lieu de http://radiofg.impek.com/fg

J'avoue que je comprenais pas pourquoi mon code marchais pas, pour une erreur toute bete... du coup le téléchargement des playlists est aussi corrompu (comme pour les fichiers textes).

Pourrais tu me donner un patch pour que j'avance là dessus.

Merci.

ghost commented 8 years ago

Probleme réglé pour les fichiers textes, et pour les playlists.

Pour les images je vais créer une lightbox en "presque" full css.

Je te poste ça dans le week end.

broncowdd commented 8 years ago

cool \o/

broncowdd commented 8 years ago

Ok, je te fais un résumé de ce que j'ai déjà fait:

intégration de tes modifs
correction des bugs dus au changement de css/balisage/structure html
changement de la vue en liste au profit d'un tableau, beaucoup plus propre et clair
pour les listes, j'ai réservé un espace pour les icônes d'action qui sont toujours visibles (plus en hover): ce point est soumis à discussion.
j'ai rétabli les nouveautés 2.3 qui avaient souffert du passage en feuilles css multiples (j'en ai ajouté du coup)

Je n'ai pas encore tout terminé, mais au final, tu devrais t'y retrouver et tes prochains ajouts devraient se faire sans douleur: d'ailleurs, il faudra qu'on reparte d'une base commune ^^

J'essaie de terminer d'ici demain, je te tiens au courant.

ghost commented 8 years ago

Ok faut voir ça par que j'ai aussi évolué un peu seulement de mon coté...

J'ai viré les markdown (fichiers .md) J'affiche les fichiers texte (txt, nfo et md) avec un $_GET['view'] avec seulement ça sur la structure avec pre un truc très basique du coup quand je les télécharge ils sont pas corrompu mais si je les affiche c'est basique et propre :)

J'ai légèrement retouché les DD (envoie de fichiers) html, css et le JS (c'est pas ma spécialité) pour traduire les erreurs.

J'ai ajouté un fichier read_m3u.html comme pour les box.

Par contre j'ai pas encore viré la lightbox pour l'instant j'aimerai le faire sur une autre version en réorganisant le menu contextuel.

J'y go...

ghost commented 8 years ago

J'ai mis sur github mes dernières modifications pour la prochaines version (il me manque juste de finir l'intégration des m3u).

Pour le reste je pense qu'on verra pour une prochaine version, il y a pas mal de changements déjà.

Qu'en penses-tu ?

broncowdd commented 8 years ago

Oui, en effet: je regarde demain et j'intègre à la 2.3 (pour voir si tout est compatible ^^)

Le 18/02/2016 20:10, EauLand a écrit :

J'ai mis sur github mes dernières modifications pour la prochaines version (il me manque juste de finir l'intégration des m3u).

Pour le reste je pense qu'on verra pour une prochaine version, il y a pas mal de changements déjà.

Qu'en penses-tu ?

— Reply to this email directly or view it on GitHub https://github.com/broncowdd/BoZoN/issues/57#issuecomment-185866796.

ghost commented 8 years ago

Juste pour info sinon le téléchargement des dossiers que je viens de tester héhé ne marche pas (erreur 403) à cause du .htaccess du dossier private où se trouve le répertoire tmp. Par ailleurs je verrais plutot tmp à la racine plutôt que dans private et dans un dossier différent pour chaque utilisateur. Allez je suis KO et malade donc bonne soirée...

broncowdd commented 8 years ago

Je regarde ça dans la journée, merci. Repose-toi bien ^^

broncowdd commented 8 years ago

Au passage, j'ai mis les js dans un dossier core/js plutôt que dans le thème pour les intégrer à la version officielle: ainsi, si un utilisateur préfère d'autres libs, il peut le faire dans son thème mais la version de base fonctionnera toujours. Je suis en train d'intégrer les nouvelles mises à jour et j'ai eu une idée (un détail) : mettre les css définissant les couleurs dans une feuille à part pour qu'on puisse simplement changer les couleurs de l'interface sans aller fouiner partout. Dans cet esprit, j'ai changé les couleurs html par des rgba pour que les nuances restent quelles que soient les modifs de la couleur de fond. Bonne journée ;-)

broncowdd commented 8 years ago

Je vais regarder aussi pour une lightbox sans jquery (j'ai trouvé un ou deux trucs).

broncowdd commented 8 years ago

Je ne vois pas l'avantage de ta modif dans share à propos des fichiers md etc. Dans ta version, les md sont affichés comme du markdown ou pas ? Ensuite, la partie où tu gères la variable GET view

if (is_in($ext,'FILES_TO_ECHO')!==false){
                      $view=htmlspecialchars(file_get_contents($f));
                      if(!isset($_GET['view'])){
                        header('Content-type: '.$type.'; charset=utf-8');
                        header('Content-Transfer-Encoding: binary');    
                        echo $view;
                    }else{
                      //affichage des fichiers text, md, et nfo comme renseigné dans le fichier listfiles.php en mode view dans le navigateur
                      echo '<html>'."\n".'<head></head>'."\n".'<body>'."\n".'<pre>'.$view.'</pre>'."\n".'</body>'."\n".'</html>';
                    }

fait la même chose que ma version

}else if (is_in($ext,'FILES_TO_ECHO')!==false){     
                        require(THEME_PATH.'/header.php');
                        echo $qrcode;       
                        echo '<pre>'.htmlspecialchars(file_get_contents($f)).'</pre>';
                        echo $call_qrcode;
                        require(THEME_PATH.'/footer.php');                      
                    }else if (is_in($ext,'FILES_TO_RETURN')!==false){
                        header('Content-type: '.$type.'; charset=utf-8');
                        header('Content-Transfer-Encoding: binary');
                        header('Content-Length: '.filesize($f));
                        readfile($f);
                    }else{
                        header('Content-type: '.$type);
                        header('Content-Transfer-Encoding: binary');
                        header('Content-Length: '.filesize($f));
                        // lance le téléchargement des fichiers non affichables
                        header('Content-Disposition: attachment; filename="'.basename($f).'"');
                        readfile($f);
                    }

si c'est un fichier à echo dans le navigateur => on l'affiche dans pre sinon, on l'envoie en brut.

Si ce que tu souhaites, c'est que le fichier soit envoyé directement en brut avec les header(), il suffit de modifier les variables 'FILES_TO_ECHO' et 'FILES_TO_RETURN', non ? A moins que tu ne veuilles pouvoir renvoyer un même type de fichier de deux façons différentes (mis en forme ou brut)... C'est ça ?

broncowdd commented 8 years ago

Pour la lightbox, je pense avoir trouvé (pour 10 ko de js pur) je tente ^^

ghost commented 8 years ago

Pour l'instant les markdown s'affiche en texte seulement, pour l'affichage non c'est pas pareil que toi puisque sans GET['view'] le fichier téléchargé aura la structure du html... pas terrible car le fichier est donc différent mais aussi je ne vois pas pourquoi afficher un fichier avec la structure du thème.

Après j'ai pas super saisi ce que tu as dis dans la deuxieme partie, mais si on affiche le fichier en brut si celui-ci a un format DOS il s'affiche correctement dans le navigateur mais si il a un format UNIX c'est illisible (les retours à la ligne ne sont pas supportés) voilà pourquoi pour le voir dans le navigateur le pre est utile. (navigateur Firefox 44 sous Win 7, pas testé sous linux, je m'en sert plus en desktop depuis 4ans).

Pour finir pour le mardown je comptais plus intégrer la class https://github.com/michelf/php-markdown (non testé, au pire ça se fork :) ) car le script mardown.php ne fonctionne pas du tout correctement. Puis bon je suis pas adepte d'utiliser des header et footer différents.

Dans un premier temps je fais au plus rapide pour avoir un truc propre sans trop de bug, le reste je vois après, c'est ma logique. D'abord on gère les détails (pas de bugs, fonctionnalités basiques et après je rajoute au fur à mesure et à mes yeux les makdown sont vraiment pas le truc le plus utile).

Gérer les mkv c'est bien plus important je trouve histoire d'alléger les DD par exemple. Pas forcément de les lire dans le navigateur (en même temps le x265 est pas supporté) mais au moins que le lien soit reconnu par vlc. Bon c'est pas super long ça à faire mais c'est pas aussi ma priorité principale.

"le markdown est aux développeurs ce que le nfo est au pirates"...

Ensuite les md ne peuvent pas être transférés... donc bon...

Pour finir j'ai réorganisé le readme.md, avec un changelog.md, et un contributors.md sur mon github, je trouve que c'est plus pratique et toi ?

broncowdd commented 8 years ago

Pour l'affichage des fichiers md, il n'utilise pas la structure du thème mais un header/footer spécial à la GitHub. La lib n'est sans doute pas optimale pour certains cas, c'est une lib que j'ai adaptée ;-)

En effet, l'idée d'ajouter un fichier contributeur est excellente ! Je bosse sur une lightbox et je finis d'intégrer au mieux tes modifs (surtout pour le problème de format que tu soulèves) Ça va mieux au fait ? A+

ghost commented 8 years ago

Tout roule :)

Bon j'ai commencé à réorganiser la traduction française qui sera la base (c'est long et pénible pour un résultat non visible pour les utilisateurs mais tellement pratique, je finirai plus tard).

J'ai suivi ta logique en mettant les js dans un dossier js à la racine, je préfère ça que de le mettre dans core mais au pire c'est un détail, vu que tu les met dans un dossier js.

J'ai finalisé l'intégration de la lecture des playlists audio m3u avec une autre logique. (http://bozon.unelio.xyz/index.php?f=156c8f76c03d56&read=m3u)

Et j'ai aussi intégré le scrolltotop de pluxml tellement pratique (ok c'était easy) :p.

Voilà je vais me mettre sur les markdown et faire les css pour les fichiers qui sont protégés par mdp quand ils sont partagés.

broncowdd / BoZoN

Design et ergonomie incohérent #57