Chamilo LMS

Actualizado por Isaac Flores hace 4 días

• Estado changed from New to Assigned * Asignado a set to Isaac Flores

#2
Actualizado por Yannick Warnier hace 1 día

• Asignado a changed from Isaac Flores to Arthur Portugal * Versión prevista changed from 1.8.6.1 to 1.8.6.2

#3
Actualizado por Yannick Warnier hace 1 día

Arthur, I think you already finished this. Could you confirm and close the task?

Hello, Which chamilo version are you using?
Can you add your CSS? In which type of question you found that?

Marko Kastelic wrote:

Sorry to inform you, but this one is not resolved. I'm attaching small image ...

The version is 1.8.7 (original),
using default stylesheets (named Chamilo, Corporate, Cosmic

problem vas described in the original post. Here's how the answer is constructed :

<p style="float:left; padding-right:4px;">
<span><input class="checkbox" type="checkbox" name="choice[2][5]" value="1" ></p></span><p>ena</p>
<p>dva</p>
<p>tri</p>

from this, i suspect, single, multiple and strict typos suffers from this problem

sorry for typos,

exercice.lib.php ( method showQuestion )

// cross-sectioned <p> && <span> elements // lines 224-225,245-246,266-267

// sample from original source from line 243-249:
<tr>
    <td colspan="3">
        <div class="u-m-answer">
         <p style="float:left; padding-right:4px;">
         <span><input class="checkbox" type="checkbox" name="choice['.$questionId.']['.$numAnswer.']" value="1" '.$selected.'></p></span>';
        $answer = api_parse_tex($answer);
        $s .= Security::remove_XSS($answer, STUDENT);
        $s .= '</div></td></tr>';

<tr> 
   <td colspan="3"> 
       <div class="u-m-answer"> 
      <p style="float:left; padding-right:4px;"> 
            <span><input class="checkbox" type="radio" name="choice[2]" value="5" >
          </p>
                 </span>

           <p>// three + lines </p> 
           <p>for (int i=0;i&lt;100;i++){</p> 
           <p>cout &lt;&lt; "test";</p> 
           <p>}</p>
       </div>
    </td>
</tr>

proposed / how it shoud be :
nesting 'answer' section to separate div, that is floated to the right, replaceing span&p tags:

<tr> 
   <td colspan="3"> 
       <div class="u-m-answer">     
      <p style="float:left; padding-right:4px;"> 
            <span><input class="checkbox" type="radio" name="choice[2]" value="5" >

                 </span>
           </p>

          <div style="float: right;">
            <p>// three + lines </p> 
            <p>for (int i=0;i&lt;100;i++){</p> 
            <p>cout &lt;&lt; "test";</p> 
            <p>}</p>
           </div> 
       </div>
    </td>
</tr>

where & what :

sequence of the lines (3 sections) : 226-228, 243-249, 264-270
that currently read as

-$answer = api_parse_tex($answer);
-$s .= Security::remove_XSS($answer, STUDENT);
-$s .= '</div></td></tr>';

should be changed as :

$answer = api_parse_tex($answer);
s .= '<div style="float: right;">;
s .= Security::remove_XSS($answer, STUDENT);
s .= '</div>';
$s .= '</div></td></tr>';

oups,
forgot to replace the position of the span and p tag (just a line above)

and ... width:95% to the answer div tag style

reminder : this one is closed but not fixed ...

12211:fb6de7d31228 Task #351 - HTML syntax corrections only.
http://code.google.com/p/chamilo/source/detail?r=fb6de7d31228ca92dc34e7f7cfe7b8d7c602bce4&repo=classic

12214:017110563d52 Task #351 - A CSS syntax correction.
http://code.google.com/p/chamilo/source/detail?r=017110563d522011fd78c5a3717ad274f58125bf&repo=classic

Ivan Tcholakov wrote:

The suggestion
$s .= '<div style="float: right;">';
moves the texts to the right (tried with Opera).

Thanks Ivan, except for the rtl, it's good enough; btw. text is moved right if the width of the div is not set ...

12229:8c9c81a1ade6 Task #351 - Code preparation for further reworks.
http://code.google.com/p/chamilo/source/detail?r=8c9c81a1ade63903d76a93d32d4ade9a40a6fc56&repo=classic

12242:2c9dac43f2d1 Task #351 - Visual fixes.
http://code.google.com/p/chamilo/source/detail?r=2c9dac43f2d1bee785dd0142c4487d95f0330d73&repo=classic