Skip to content

Made fixes to accessibility and returning back to a page after a time up or answered questions - #4

Open
mike-st wants to merge 67 commits into
simondate:masterfrom
mike-st:master
Open

Made fixes to accessibility and returning back to a page after a time up or answered questions#4
mike-st wants to merge 67 commits into
simondate:masterfrom
mike-st:master

Conversation

@mike-st

@mike-st mike-st commented Oct 23, 2018

Copy link
Copy Markdown
Contributor

No description provided.

@simondate

Copy link
Copy Markdown
Owner

@mike-st , thank you so much for creating such a detailed PR for a component that I haven't touched in almost three years.

When I get a chance I will review it and merge it in. But thanks so much!

@mike-st

mike-st commented Oct 25, 2018

Copy link
Copy Markdown
Contributor Author

Thanks so much Simon! Great work again on this one. Saved my hid for a client job I was working on.

I had slightly over looked some things in the last push, without properly testing accessibility and when a person navigates back to the timed MCQ from the menu when it has been previously answered or timed out. Fixes should help make it more accommodating going on.

@simondate

Copy link
Copy Markdown
Owner

Likewise your menu was exactly what one of my clients requested. Pretty cool coincidence 😄

I have also made a few improvements on my fork of it. So when I'm finished I'll create a PR too!

As for this one I'll wait til my work dies down a bit and I will test locally before merging.

@mike-st

mike-st commented Oct 25, 2018

Copy link
Copy Markdown
Contributor Author

Sounds good and that is great I could help out with the menu for you too. Thanks for making some improvements too. Looking forward to checking them out, much appreciated. My strengths are a little more CSS/LESS so hopefully my js wasn't too messy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants