[MDL-66503] Scorm player is really narrow in new window on classic theme - Moodle Tracker

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Major
Resolution: Fixed
Affects Version/s: 3.7.1
Fix Version/s: 3.7.4, 3.8.1
Component/s: SCORM
Labels:

Testing Instructions:
Hide

Please test on Boost and Classic themes.

In a course create a new scorm activity.

Upload a scorm package. You can find some to download from here.

Under "Appearance" set "Display package" to "New window"

Click "Save and display"

Enter the scorm activity.

Check that the scorm activity takes up the full window.
Show
Please test on Boost and Classic themes. In a course create a new scorm activity. Upload a scorm package. You can find some to download from here . Under "Appearance" set "Display package" to "New window" Click "Save and display" Enter the scorm activity. Check that the scorm activity takes up the full window.
Affected Branches:
MOODLE_37_STABLE
Fixed Branches:
MOODLE_37_STABLE, MOODLE_38_STABLE
Pull from Repository:
git://github.com/abgreeve/moodle.git
Pull 3.7 Branch:
~~MDL-66503~~-37
Pull 3.7 Diff URL:
https://github.com/abgreeve/moodle/compare/fcc93935be...MDL-66503-37
Pull 3.8 Branch:
~~MDL-66503~~-38
Pull 3.8 Diff URL:
https://github.com/abgreeve/moodle/compare/773b1f738f...MDL-66503-38
Pull Master Branch:
~~MDL-66503~~-master
Pull Master Diff URL:
https://github.com/abgreeve/moodle/compare/8cf4c97217...MDL-66503-master

Story Points:
3
Sprint:
Activity Sprint 2

Description

With the Classic theme enabled:

Create a new scorm activity
- Under "Appearance" set "Display package" to "New window"
Click "Save and display"
Enter the scorm activity

In boost, the player takes up the full width of the window, in classic it takes up less than half of it.

To test, I used the scorm package available here

Attachments

Options
- Sort By Name
- Sort By Date
- Ascending
- Descending
- Thumbnails
- List
- Download All

Attachments

Confirmed fixed.png
104 kB
18/Dec/19 9:25 AM

Activity

People

Assignee:: Adrian Greeve

Reporter:: cameron1729

Peer reviewer:: Dan Marsden

Integrator:: Jake Dallimore

Tester:: Gladys Basiana

Participants:: Adrian Greeve, cameron1729, CiBoT, Dan Marsden, Gladys Basiana, Ian Fogarty, Jake Dallimore, Lee Goldsworthy, Peter Dias

Component watchers:: Damyon Wiese, Dan Marsden, Matteo Scaramuccia, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias

Votes:: 1 Vote for this issue

Watchers:: 8 Start watching this issue

Dates

Created:: 27/Aug/19 1:04 PM

Updated:: 02/Jan/20 3:02 PM

Resolved:: 20/Dec/19 6:24 PM

Fix Release Date:: 13/Jan/20

Time Tracking

Estimated:

0m

Remaining:

0m

Logged:

2h 52m