automne-cms / automne

Intuitive and powerful CMS based on Apache, PHP, MySQL
https://www.automne-cms.com
14 stars 9 forks source link

Message de notification et compatibilité IE7 #77

Closed emmanuelVentura closed 4 years ago

emmanuelVentura commented 11 years ago

Je travaille actuellement sur la différenciation des messages d'Automne afin de modifier les messages d'erreur pour les rendre plus reconnaissable. Cette démarche me permet de voir qu'Automne gère l'arrondi du message avec des div imbriqués et un sprite pour les images.

Je souhaiterais me passer de cette "antiquité" et utiliser "border radius". La compatibilité IE7 n'en est pas perdue, ce navigateur n'affichera simplement pas de bords arrondis.

Ne voulant pas remettre en cause le travail de fond de ma contribution, je souhaiterais connaitre votre point de vue afin de m'aligner sur la politique du CMS en matière de compatibilité attendue.

Merci

harmo commented 11 years ago

La mode n'est pas revenue aux coins carrés depuis W8 ? :trollface:

DirtyF commented 11 years ago

Tant qu'à faire, autant standardiser le code, quitte à ce que IE7 n'affiche pas les bords arrondis, ce n'est pas bien grave. Pensons au futur plutôt qu'au passé.

Le dimanche 24 février 2013, harmo a écrit :

La mode n'est pas revenu aux coins carrés depuis W8 ? [image: :trollface:]

— Reply to this email directly or view it on GitHubhttps://github.com/automne-cms/automne/issues/77#issuecomment-14012612.

emmanuelVentura commented 11 years ago

Moi je veux bien, c'est juste une confirmation que j'attendais. J'ai supprimé le code suivant

var boxtpl = ['<div class="msg ', msgType ,'">',
'<div class="x-box-tl"><div class="x-box-tr"><div class="x-box-tc"></div></div></div>',
'<div class="x-box-ml"><div class="x-box-mr"><div class="x-box-mc"><strong>', title, '</strong>', message, '</div></div></div>',
'<div class="x-box-bl"><div class="x-box-br"><div class="x-box-bc"></div></div></div>',
'</div>'].join('');

Pour le remplacer par celui-ci

var boxtpl = ['<div class="msg ', msgType ,'">',
'<strong>', title, '</strong>', message, '',
'</div>'].join('');

Et la feuille de style qui va avec. C'est juste que je trouverai dommage de voir mon Pull Request refusé pour ça, car du coup je ne me contente pas d'ajouter quelque chose, mais je modifie quelque chose existant.