Skip to content

Conversation

@mike-st
Copy link
Contributor

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

No description provided.

@simondate
Copy link
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
Copy link
Contributor Author

mike-st commented Oct 25, 2018

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
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
Copy link
Contributor Author

mike-st commented Oct 25, 2018

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