Issue Details (XML | Word | Printable)

Key: CONTRIB-534
Type: Sub-task Sub-task
Status: Resolved Resolved
Resolution: Fixed
Priority: Minor Minor
Assignee: Olli Savolainen
Reporter: Olli Savolainen
Votes: 0
Watchers: 0
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Non-core contributed modules
CONTRIB-528

Clean up any inline javascript, CSS & HTML

Created: 26/Jun/08 11:07 PM   Updated: 20/Nov/08 03:16 AM
Component/s: Add a project here
Affects Version/s: 2.0
Fix Version/s: None

Participants: Olli Savolainen
Security Level: None
Resolved date: 20/Nov/08
Affected Branches: MOODLE_20_STABLE


 Description  « Hide
Move to appropriate libraries and themes; research what is the optimal approach with these

edit.php's own javascript functionality is currently included in the body of the page and this works ok and validates.

Stylesheets need to be added to the theming system, but some of them are theme-independent since they define the layout of quiz UI without which Quiz would be broken. It would seem reasonable not to force the user's browser to download all of these styles unless they actually use the edit tabs. is this possible in moodle currently?

Hiding the repaginating dialog, only to users with javascript, properly requires loading the extra stylesheet using js in head of the page. I currently have a hack for this (in lib/weblib.php), but some mechanism should be provided to bring this to the head section of html produced by only edit.php. Also it is currently non-yui javascript atm, but assuming that moodle is going to use the newest yui lib in 2.0, this can easily be made into YUI javascript.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
There are no comments yet on this issue.