Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #2054: Le site n'est pas compatible IE9 #2263

Merged
merged 2 commits into from Mar 6, 2015
Merged

Fix #2054: Le site n'est pas compatible IE9 #2263

merged 2 commits into from Mar 6, 2015

Conversation

ghost
Copy link

@ghost ghost commented Feb 9, 2015

Q R
Correction de bugs ? oui
Nouvelle Fonctionnalité ? non
Tickets (issues) concernés #2054

Sous IE 9, la sidebar ainsi que le footer s'affiche correctement.

lt = "inférieur à", c'est lte = "inférieur ou égal à"
@SpaceFox
Copy link
Contributor

SpaceFox commented Feb 9, 2015

La vraie QA ici, c'est de vérifier que rien n'est pété sous les navigateurs utilisés. IE9 l'est tellement peu que le site ne pourrait pas marcher du tout avec que ça ne serait même pas un vrai problème.

@@ -43,6 +43,9 @@ textarea {
color: #000;
padding: 0.2em 0;
text-align: center;
position: fixed;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ça ne devrait pas être dans le fichier only-ie.scss ?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Regarde le nom de la règle (ligne 40). Cette div n'apparait que quand c'est IE.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bah du coup tu peux transférer cette règle dans le fichier only-ie.scss je pense ! :)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tu penses quoi de transférer ça dans le fichier only-ie.scss ?

@Situphen
Copy link
Member

Il n'y a pas l'air d'avoir de changements avec l'affichage actuel sur Firefox 35.

@ghost
Copy link
Author

ghost commented Feb 10, 2015

Opera semble avoir le même bug que IE9 sur la sidebar.

Imgur

@SpaceFox
Copy link
Contributor

Je ne vois pas le bug ici ?

D'une manière générale, les versions récentes d'Opera utilisent Blink, donc le moteur de Chrome. Donc ça ne nécessite plus de test spécifique.

@ghost
Copy link
Author

ghost commented Feb 10, 2015

Sur IE j'ai corrigé un faux bug d'affichage. ^^" Je corrige ma PR.

EDIT : Non tout est Ok. Par contre sur IE9 la sidebar descend tout en bas.

Imgur

@Eskimon Eskimon added S-BUG Corrige un problème C-Front Concerne l'interface du site labels Feb 10, 2015
@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling abf584a on A-312:fix-2054-ie9 into ed7d9d0 on zestedesavoir:dev.

@ghost
Copy link
Author

ghost commented Feb 14, 2015

Petite correction du footer :

Imgur

@firm1
Copy link
Contributor

firm1 commented Feb 24, 2015

Cette branche est actuellement déployée sur : http://vps137741.ovh.net/

En ce qui me concerne j'ai testé sous IE9 et ça passe très bien.
Sous Firefox 39 aussi ça marche pas mal.

Quelqu'un d'autre pour vérifier les autres Navigo ?

@firm1
Copy link
Contributor

firm1 commented Feb 24, 2015

Testé sur Chrome (PC et mobile), ça se comporte normalement.

Pour moi QA OK. à merger

@Situphen
Copy link
Member

A part mon commentaire, c'est OK pour moi aussi !

@SpaceFox
Copy link
Contributor

SpaceFox commented Mar 5, 2015

@A-312 tu peux prendre en compte le commentaire de @Situphen ?

@ghost
Copy link
Author

ghost commented Mar 6, 2015

@SpaceFox Ce n'est pas possible le css, n'est pas inclus pour toutes les versions d'IE.

@Situphen
Copy link
Member

Situphen commented Mar 6, 2015

Bon bah dans ce cas là on peut merger ! ;)

GerardPaligot added a commit that referenced this pull request Mar 6, 2015
Fix #2054: Le site n'est pas compatible IE9
@GerardPaligot GerardPaligot merged commit 21e62a7 into zestedesavoir:dev Mar 6, 2015
@ghost ghost deleted the fix-2054-ie9 branch March 16, 2015 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site S-BUG Corrige un problème
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants