-
Notifications
You must be signed in to change notification settings - Fork 61
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
accessible/accessable option removed? #69
Comments
This was probably removed in #35 but please confirm. |
We actually removed this in 05e8d9f, to fix a couple of other bugs (#30 and #21). In that commit, we simplified the plugin and removed alot of complicated logic for "boxing in" the highlighted element (we fitted a transparent overlay over the highlighted element, and used box-shadows combined with a fullscreen dark overlay). The intention was to re-introduce the accesibility feature at some point (see #42), but we never got around to it. We've also been planning to do a full rewrite ( With the way it's setup now, there's no way for us to re-implement this feature without changing most of the plugin's code, so we'd probably be better off doing the full rewrite and implementing it at that point. |
I understand, that's what I thought. |
No worries, sorry I can't help any further at this point in time. |
Hi guys,
Just wanted to check with you if you got rid of the accessible option? I have some form element I'd like to have accessible for user when walkthrough is displayed and I wonder if there's a way to do that with current version?
Thanks
The text was updated successfully, but these errors were encountered: