Skip to content
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

Tab sequence can not be controlled in forms which include prettycheckable #47

Open
aimfeld opened this issue Aug 28, 2013 · 9 comments
Open

Comments

@aimfeld
Copy link

aimfeld commented Aug 28, 2013

I use forms with some prettycheckable elements and some other elements. Tabbing through my form used to work, but now I need to set a tabindex value for every element to control the tab sequence. However, as soon as I set the tabindex value on my form elements, prettycheckable elements are not included in the tab sequence anymore (tested with Chrome). There is just no way to control the tab sequence in forms which include prettycheckable elements...

@arthurgouveia
Copy link
Owner

@aimfeld thanks for pointing this out. Do you have a live URL where I can check this?

Also, is this something you would be able to fix on the plugin?

Thanks.

@aimfeld
Copy link
Author

aimfeld commented Aug 28, 2013

I set up a jsFiddle to test this: http://jsfiddle.net/aZLSA/2/

If you put the cursor on the input field and then tab through the elements, you'll see that the tab sequence defined by the tabindex values is not working.

I had a look at your code but I don't know how to fix this..

@arthurgouveia
Copy link
Owner

I see the issue.

I won't be able to have any time to work on this until mid September.

If you want to give it a shot, you would need to scan the cloned input for it's attributes and if exists a "tabindex", it should be cloned to the anchor tag that works as the style input.

@markushausammann
Copy link
Contributor

Did you solve this too with 2.0 ?

@markushausammann
Copy link
Contributor

Doesn't seem so, looking at the code.

@arthurgouveia
Copy link
Owner

Not yet. I'll work on having all properties of the input cloned to the wrapping div.

@arthurgouveia
Copy link
Owner

@aimfeld maybe you can clone the tabindex attribute from the input/label and set it on the anchor tag that "replaces" the input.

Are you willing to make a PR with this?

@aimfeld
Copy link
Author

aimfeld commented May 12, 2014

We have experienced a variety of problems with pretty checkable. We needed to introduce quite a few javascript and css special cases for pretty checkable in our codebase. We are currently in a frontend refactoring process (twitter bootstrap 3) and we will evaluate a css-only solution for radio buttons and checkboxes. If we decide to give pretty checkable another try, I'll let you know. It's hard to let pretty checkable go, it really does look pretty :).

@arthurgouveia
Copy link
Owner

Next time I touch this plugin, I'm thinking of getting rid of the images on the background and simply have the anchor tag look like the checkable container and add a :before pseudo selector to style the checkmark on both cases.

Second thing is getting rid of the jQuery dependency.

Let me know.

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

No branches or pull requests

3 participants