Replace table-based display with jsTree for accessibility
|Participant||Role||Time Spent||Comments||Latest Comment|
|Author||1h 33m||24||Yes. I plan to get them all before the final commit.|
|Reviewer - 95% reviewed||29m||7||So when options.hasOwnProperty('onlyOwnedFolders') evalua...|
|Reviewer - 97% reviewed||24m||3||There are a few todo's in the different files. Remember t...|
|Reviewer - 91% reviewed||7m|
|Reviewer - 97% reviewed||46m||11||remove the comment if no needed|
|Reviewer - Complete||36m||7||The nesting depth in googleDriveLti.js will make it hard ...|
|Reviewer - 0% reviewed||0m|
Inspired by GOOGLE-115, the GDL tool's table-based UI is replaced by jQuery plug-in jsTree.
Gonzalo's suggestion was to use a jQuery plug-in called jsTree (see link below), which is a full-featured tree, features ARIA-compliant accessibility, and was also planned for use with the Sakai resources UI. Basically, replace all the locally-written code that displays the folder/file tree with simpler code that calls this jQuery plug-in to display it instead. This will require some moderate rewriting of the GDL UI code, but it will also simplify the project's codebase.
The code in this review is functional and covers the basic GDL UI requirements. Some of the other features (e.g. searching, adding new items to folders, etc.) have been moved to other JIRA issues and are not complete here. This code could stand some cleaning up and clarification. I hope the review comments received here will help me concentrate on what others think needs to be fixed.
Branches in review
Issues Raised From Comments