WordPress.org

Support

Support » Plugins and Hacks » Theme My Login » [Resolved] Tabindex added in 6.2 and removed in 6.2.3

[Resolved] Tabindex added in 6.2 and removed in 6.2.3

Viewing 6 replies - 1 through 6 (of 6 total)
  • Hey there!

    I also would like to know why you, Jeff Farthing, removed the patch contributed by Marco over a year ago.

    I tried to change the register-form.php by my own following the lines in Marco’s patch (see: http://wordpress.org/support/topic/correct-tabindex?replies=2).

    But it seems Theme My Login doesn’t accept this changes, i. e., fields added with CIMY – plugin are still skiped… :/

    Any further suggestions?!

    Thanks in advance & regards!

    Chipy

    Plugin Author Jeff Farthing

    @jfarthing84

    Because browsers can handle tabindexing on their own. This is why I, Jeff Farthing, decided to remove tabindexes.

    @ Jeff Farthing:
    I’m sorry, but you’re NOT right at all.
    I tried different browsers (Opera, Firefox, Konqueror, …), but it didn’t handle tabindexing on their own in none of them.

    Inspirited by the lines of Marco Cimmino’s patch for the “Theme My Login” – Plugin, which he contributed over a year ago, I finally found the solution for the skipping “troubles”.

    Instructions:
    1. Open ~/wp-content/themes/[the_name_of_your_theme]/register-form.php .

    2. Change line 19 from:
    <input type="text" name="user_login" id="user_loginthe_instance(); ?>" class="input" value="the_posted_value( 'user_login' ); ?>" size="20" />

    → to:
    <input type="text" name="user_login" id="user_loginthe_instance(); ?>" class="input" value="the_posted_value( 'user_login' ); ?>" size="20" tabindex="10" />

    2. Change line 24 from:
    <input type="text" name="user_email" id="user_emailthe_instance(); ?>" class="input" value="the_posted_value( 'user_email' ); ?>" size="20" />

    → to:
    <input type="text" name="user_email" id="user_emailthe_instance(); ?>" class="input" value="the_posted_value( 'user_email' ); ?>" size="20" tabindex="10"

    3. Open ~/wp-content/plugins/theme-my-login/modules/custom-passwords/custom-passwords.php .

    4. Change line 31 from:
    <input autocomplete="off" name="pass1" id="pass1the_instance(); ?>" class="input" size="20" value="" type="password" />

    → to:
    <input autocomplete="off" name="pass1" id="pass1the_instance(); ?>" class="input" size="20" value="" type="password" tabindex="10" />

    5. Change line 32 from:
    <input autocomplete="off" name="pass2" id="pass2the_instance(); ?>" class="input" size="20" value="" type="password" />

    → to:
    <input autocomplete="off" name="pass2" id="pass2the_instance(); ?>" class="input" size="20" value="" type="password" tabindex="10" />

    As you can see, I added four times the SAME (NOT DIFFERENT!!!) tabindex (tabindex=”10″) for the first four fields.

    I am not sure, if it is the really right way, but anyway:
    Finally, it works as it should and as I want it to do!!!

    Now it’s possible to skip with the tabulator from one field to another in the right order and without any problems.

    I hope to help the community in sharing my solution with all of you.

    Regards from Germany!
    Chipy

    P.D.: So, Jeff, maybe you finally can change what a lot of the users, who are using your plugin together with the one of Marco for even more functionality, ask you to do!

    Plugin Author Jeff Farthing

    @jfarthing84

    That is because the CIMY Extra Fields plugin is adding tabindexes. Go figure.

    That is because the CIMY Extra Fields plugin is adding tabindexes. Go figure.

    Jeff don’t hide behind a finger, I can still see you.
    WordPress default registration form was adding tabindex till v3.4 too, I am not like an alien that was doing this all alone, come on.

    I just noticed now that they removed them together with WordPress 3.5, fine, I will remove them too.

    But don’t point fingers like you heard something crazy while it is not.
    Anyway close this, no really point to establish a collaboration with you since you don’t want to.

    Plugin Author Jeff Farthing

    @jfarthing84

    I’m not sure I get what you’re saying. I’m not hiding. I’m simply stating the facts. The whole nature of this post from the start was condescending toward me.

Viewing 6 replies - 1 through 6 (of 6 total)
  • The topic ‘[Resolved] Tabindex added in 6.2 and removed in 6.2.3’ is closed to new replies.
Skip to toolbar