Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-39446

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

          Details

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

            1. Go to create a calculated question. Make sure the Answer formula box is a good size.

            2. Ditto, Calculated simple qtype.

            master, and stable branches need to be tested separtately, since the forms have changed

            Show
            1. Go to create a calculated question. Make sure the Answer formula box is a good size. 2. Ditto, Calculated simple qtype. master, and stable branches need to be tested separtately, since the forms have changed
          • Affected Branches:
            MOODLE_23_STABLE
          • Fixed Branches:
            MOODLE_23_STABLE, MOODLE_24_STABLE
          • Pull from Repository:
          • Pull Master Branch:
          • Pull Master Diff URL:

            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?

              Gliffy Diagrams

                Attachments

                  Activity

                  Ascending order - Click to sort in descending order
                  Hide
                  Permalink
                  timhunt Tim Hunt added a comment -

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

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

                  Reviewing the master diff. Using standard methods. Seems fine

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

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

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

                  Thanks Colin. Submitting for integration now.

                  Show
                  timhunt Tim Hunt added a comment - Thanks Colin. Submitting for integration now.
                  Hide
                  Permalink
                  poltawski 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
                  poltawski 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
                  Permalink
                  poltawski Dan Poltawski added a comment -

                  Integrated to master, 24 and 23, thanks.

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

                  This is working as expected.

                  Tested for 2.3, 2.4 and master.

                  Test passed.

                  Show
                  rwijaya Rossiani Wijaya added a comment - This is working as expected. Tested for 2.3, 2.4 and master. Test passed.
                  Hide
                  Permalink
                  stronk7 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
                  stronk7 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:
                        Fix Release Date:
                        13/May/13