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

editpdf popup should be dragable

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.6
    • Fix Version/s: 2.6
    • Component/s: Assignment
    • Labels:
    • Testing Instructions:
      Hide
      1. Open the edit pdf window
      2. Verify it does opens near the top of the page
      3. Verify you can drag the titlebar to move the window
      4. Move the window around a bit
      5. Use some drawing tools and verify that the annotations are drawn in the correct place.
      Show
      Open the edit pdf window Verify it does opens near the top of the page Verify you can drag the titlebar to move the window Move the window around a bit Use some drawing tools and verify that the annotations are drawn in the correct place.
    • Affected Branches:
      MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-42401-master

      Description

      My instinct when the popup appeared was to drag it in a better position.

      I see the reason for that as 2 reasons:

      1. The pdf was 1/4 of the way down my screen, and it didn't all fit on the screen, so i wanted to move it higher
      2. Those JS dialogues all have a 'bar' at the top, which instinctively makes me drag it. As we discussed in the office, it might be clearer if the drag bar was different on non-draggable dialogues

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              damyon Damyon Wiese
              Reporter:
              poltawski Dan Poltawski
              Peer reviewer:
              Jason Fowler
              Integrator:
              Dan Poltawski
              Tester:
              Sam Hemelryk
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                18/Nov/13