Chamilo-lms: correctif fill_blanks.class.php

[alexiye]

Current behavior / Resultado actual / Résultat actuel

-corriger ligne 1279
feedback-red devient feedback-green
-corriger ligne 1290
il manque un point virgule après feedback-question
Après la correction :

Si on veut garder le contour on peut aussi avoir :

à vous de voir...

Expected behavior / Resultado esperado / Résultat attendu

Est-ce que j'ai gagné un mars ?

Steps to reproduce / Pasos para reproducir / Étapes pour reproduire

Chamilo Version / Versión de Chamilo / Version de Chamilo

[jmontoyaa]

This part of the code has been change, it works well for me. Can you try again?
Better if you can make an example of the error here:

https://11.chamilo.org/

login/pass: admin/admin

[alexiye]

https://11.chamilo.org/main/exercise/exercise_result.php?cidReq=FILLBLANKS&id_session=0&gidReq=0&gradebook=0&origin=&exe_id=613&learnpath_id=0&learnpath_item_id=0&learnpath_item_view_id=0

Quand on fait prévisualiser l'exercice, on ne voit pas le menu déroulant [x|y|z]

[jmontoyaa]

I sent a fix for the preview, your exercise should contain only one "|" not double "||"

henry||HENRY||Henry

[alexiye]

double || c'est quand il y a plusieurs reponses possibles non ?

Alexandre Bigan

Le 14 mars 2018 à 13:26, Julio Montoya [e-mail] a écrit :

I sent a fix for the preview, your exercise should contain only one "|" not double "||"

henry||HENRY||Henry

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

[jmontoyaa]

Oui, tu as raison. j'ai parlé avec @baelmyhu l'expert de fill in blanks et il m'a raconté comment ca marche.

Je peux réproduire l'erreur maintenant.

[jmontoyaa]

J'ai corrigé l'erreur.

[alexiye]

Ne manque-t-il pas un point virgule à la ligne 1290 pour éviter d'avoir la correction sur deux ou trois lignes :

En rajoutant ce point virgule, j'ai la réponse sans saut à la ligne avant le blanc et après le blanc.

[jmontoyaa]

Ce changement n'est pas un CSS valide ...

C'est à cause de "display: block;"

.feedback-question {
    margin-top: 5px;
    margin-bottom: 5px;
    // display: block;
}

[AngelFQC]

This style for question was created for ofaj branch and the images should not be here but was added to 1.11.x here https://github.com/chamilo/chamilo-lms/commit/531f5aa24e3f66c730bf48d2cd78f8e950c3364c#diff-8b186ddd6e577c60ef19e31036be3b37R7330
I sent a minor style change

[AngelFQC]

@aragonc sería mejor quitar las imágenes que parecen radiobuttons o mejorar el color para que vaya con el tema de chamilo. Aunque inicialmente estas imágenes no estaban en 1.11.x

[jmontoyaa]

Si se hacen cambios confirmarlos con @NicoDucou para que se integren bien en ofaj_dev
La idea es que todo lo que va a 1.11.x será pasado a ofaj_dev.

[alexiye]

Je suis désolé mais j'ai de nouvelles erreurs que je n'arrive pas à comprendre.
Celles-ci sont visibles dans le lien suivant, puis affichage des résultats :

https://11.chamilo.org/main/exercise/exercise_result.php?cidReq=LEPASSECOMPOSECE2&id_session=0&gidReq=0&gradebook=0&origin=&exe_id=626&learnpath_id=0&learnpath_item_id=0&learnpath_item_view_id=0

Quand plusieurs réponses sont possibles j'utilise ||
Cela fonctionne pour je suis [venu||venue] mais des erreurs surviennent avec tu es [allé||allée] et vous êtes [allés||allées] !

Si je change le verbe aller par le verbe partir, cela donne [parti||partie] et là cela fonctionne. L'erreur proviendrait-elle du 'é' dans [allé||allée] ?

[alexiye]

J'ai vu que cela a été corrigé, merci !

[aragonc]

@jmontoyaa creo que ahora es un icono fijo, se tendría que ver como hacer el cambio para no afectar a ofaj_dev, pensé que no se integraría a Chamilo? @NicoDucou el ofaj_dev esta actualizado con la 1.11.x actual?

[jmontoyaa]

Lo mejor sería hacerlo por css, cosa que el cambio afecta a base.css pero es modificable en el estilo de ofaj.

El portal de ofaj está parcialmente actualizado a 1.11.x.

[aragonc]

Lo envié

a la rama 1.11.x quedaría asi, en ofaj faltaría revisarlo, aunque creo que esos iconos están en la carpeta del theme que ofaj usa.
Por cierto me olvide colocar el número de issue en el commit

[jmontoyaa]

El "check" debería estar en verde y la "cruz" rojo.

[aragonc]

He mejorado la vista, separando la respuesta correcta del sistema con la del usuario, adicionalmente las imágenes de los check y error verde o rojo según corresponda.

[alexiye]

Le style ne s'applique pas tout le temps ! Sinon c'est super.
https://11.chamilo.org/main/exercise/exercise_show.php?cidReq=LACCORDENGENREDELADJECTIFQUALIFIC&id_session=0&gidReq=0&gradebook=0&origin=&action=qualify&id=653

Quand on utilise les blancs avec le menu déroulant le style ne s'applique pas.

[jmontoyaa]

Buena mejora, pero en el screenshot que muestras se destaca más lo gris de todo el texto. Tal vez bajarle un poco el gris y hacer llamativo el rojo y verde asi:

[aragonc]

Cambio de color aplicado... también a listas despegables.

[alexiye]

Bravo !

[jmontoyaa]

Me parece que ese verde es poco legible. Tratar de ponerle un fondo verde y texto claro o obscuro.
Por ejemplo, el texto en azul claro en tu screenshot esta muy bien!

[alexiye]

Le style ne s'applique pas quand on peut avoir plusieurs solutions possibles en utilisant "||"

[alexiye]

:+1:

[jmontoyaa]

No es lo que solicitaba, pero igual no se ve mal.

[ywarnier]

This looks good to me now (and I discussed what "lo que solicitaba" means with Alex :sweat_smile:).