Moodle
  1. Moodle
  2. MDL-39446

The simple calculated question text box only shows 12 characters at once

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3.6
    • Fix Version/s: 2.3.7, 2.4.4
    • Component/s: Questions
    • Labels:
    • Rank:
      50106

      Description

      The simple calculated question text box only shows 12 characters at once this can be an issue when the questions are complicated formulas.

      Can this be expanded, perhaps to fill 100% of the width of the form?

        Activity

        Hide
        Tim Hunt added a comment -

        Colin, please could you peer review this small patch. Thanks.

        Show
        Tim Hunt added a comment - Colin, please could you peer review this small patch. Thanks.
        Hide
        Colin Chambers added a comment -

        Reviewing the master diff. Using standard methods. Seems fine

        Show
        Colin Chambers added a comment - Reviewing the master diff. Using standard methods. Seems fine
        Hide
        Colin Chambers added a comment -

        Seems fine to me. One line fix. No other changes

        Show
        Colin Chambers added a comment - Seems fine to me. One line fix. No other changes
        Hide
        Tim Hunt added a comment -

        Thanks Colin. Submitting for integration now.

        Show
        Tim Hunt added a comment - Thanks Colin. Submitting for integration now.
        Hide
        Dan Poltawski added a comment -

        The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

        TIA and ciao

        Show
        Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
        Hide
        Dan Poltawski added a comment -

        Integrated to master, 24 and 23, thanks.

        Show
        Dan Poltawski added a comment - Integrated to master, 24 and 23, thanks.
        Hide
        Rossiani Wijaya added a comment -

        This is working as expected.

        Tested for 2.3, 2.4 and master.

        Test passed.

        Show
        Rossiani Wijaya added a comment - This is working as expected. Tested for 2.3, 2.4 and master. Test passed.
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Did you think this day was not going to arrive ever?

        Your patience has been rewarded, yay, sent upstream, thanks!

        Closing...ciao

        Show
        Eloy Lafuente (stronk7) added a comment - Did you think this day was not going to arrive ever? Your patience has been rewarded, yay, sent upstream, thanks! Closing...ciao

          People

          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: