search

lzear / LeekWars_Kikimeter

5 stars 3 forks source link

alert sur logs d'erreur #20

Closed yLark closed 10 years ago

yLark commented 10 years ago

Afficher une notification du type window.alert() avec le contenus des log d'erreur debugE(). Pas besoin de scroller toute la page pour voir si on as eu des logs d'erreur. Ajout d'une option pour activer/désactiver cette fonctionnalité.

abourdin commented 10 years ago

Je verrais plutôt ça tout en haut du rapport dans un bloc de ce genre : http://getbootstrap.com/components/#alerts-dismissible

Avec les warnings dans un bloc, les erreurs dans un autre.

Parce qu'une window.alert, en plus d'être relativement pénible, ça fait un peu cheap. Et si il y a beaucoup d'erreurs, ça ne ressemblerait à rien.

yLark commented 10 years ago

Ou juste une petite notification en tête du rapport : suggestion error warning

C'est certain que c'est pénible le window.alert.

abourdin commented 10 years ago

On peut toujours aller pour les petites notifications, et ajouter les textes d'alerte/warnings dans des box en haut qu'on rend activables/désactivables dans la config comme le reste.

yLark commented 10 years ago

Tout à fait. Je code un petit truc.

Foudge commented 10 years ago

Sous forme de notification, c'est pas mal je trouve. On pourrait même y ajouter les plantages. Je sais qu'ils apparaissent ailleurs, mais c'est pas forcément très visible, et quitte à regrouper tous les problèmes d'IA dans un coin, autant y mettre les 3 infos.

yLark commented 10 years ago

Les plantages apparaissent dans les erreurs, donc ils devraient rester bien visible (à moins d'abuser des debugE).

abourdin commented 10 years ago

Au pire il suffit de détecter le "a planté" pour éventuellement mettre ces messages en gras avec une petite icône pour les mettre en évidence.

yLark commented 10 years ago

Commit : https://github.com/Zear06/LeekWars_Kikimeter/commit/6e5252a48220f0b57e418193cddefbaa44891932