Week – 4

Progress:

During this week, I was busy with modifying code to implement my mentor Greg’s suggestions. So in this report I have included his suggestions with my comments.

– Provide feedback after clicking Update Now for admins
–>Done.

– Replace all hard coded language with language tokens and move language into module.sql. Prefix all your language token with “ols_” or something like that so its east to identify the language in the language_text table.
–>Done. I have chosen “ol_” as prefix for tokens.

– OL Configuration could be the Current Configuration. (shouldn’t need two screen for that)
–>Done.

– still need to figure out how to get all the OL records into the db, in addition to the 50
–>Done. It takes some time so I need to optimise it but it is working smoothly.

– Allow students to search OL, but only preview the content, not download or import
–>Done.

– I like the modal dialog used for the preview. But, it is not accessible. I’m unable to access it by keyboard
–>Done. I removed third party API and used a simple popup window. I read the accessibility issues of the dialog so I decided to dump it.

– The linked result titles are not keyboard accessible. Search results should be all navigable by keyboard
–>Done. I removed Accordion of jquery because it has problem of keyboard navigation and used CSS instead. So keyboard navigation works like a charm.

– For instructors, make search OL a child of content, like Import/Export is.
–>Done.

– the next/previous images need alt text. and they could be duplicated at the top of the results
–>Done.

– avoid using onchange() to reset the maximum record. It will be an accessibility problem. Instead try scripting to the enter key, or include a submit button.
–>Done.

– The search form should be included at the top of the search results.Currently to only way back to the form is using the back button.
–>Done.

– I am able to import a downloaded cartridge, but not import directly from search results. I get a “The file does not appear to be a valid ZIP file.” error, which seems to be incorrect. Maybe download the cartridge into a temp directory, like content/import/ then take the downloaded file and import through a copy of the import content form. Opening a socket may be another option. You’ll need to experiment.
–>In progress. I think I will have to give my time on this issue and try to find where the problem is. Many of the OL packages are corrupt and some packages which are not corrupt gives an error of some xml file when I tried to import manually in ATutor. So I will consider this issue in the coming days.