Unusual html for twitter sign in button

try this on your css stylesheet as it works on our site:

form#user-login .item-list ul li {
list-style: none;
padding: 0px;
margin:0px;
}

I agree. A specific class would be very helpful in my case as well. There are other items that are affected when I try to theme the button.

To be more specific, in the standard sign in block, the language "Create new account" and "Request new password" are also affected by

form#user-login .item-list ul li

Just chiming in - it would be really, really great to have a css class for the sign in button. Would make life a lot easier when theming.

I figured out a way to select the sign-in button.

a[href="/twitter/redirect"] {css styling info goes here}

I've managed to use that to separate it from the create new account and request new password links so I can position it independently. Seems to work.

Hi
There have been some work done to help out front end guys http://drupalcode.org/project/twitter.git/commit/02049c7 fixing [#736698 Test it, try overriding, post your comments. We are open for suggestions and patches regarding the signin button.

This patch adds class attribute to the link-button and an option in the admin menu to configure the button also a text (instead an image).

The CSS class is named 'twitter-signin-menuitem'. Other extra class 'twitter-signin-menuitem-just-text' or 'twitter-signin-menuitem-image' indicates if the link-button is text or image.

I agree, lists for this should be optional (maybe can be used by other modules like Drupal for Facebook to place its button). In other hand an option to decide where to put the button (before the inputs or after) and a custom image to upload one would be great too.

Patch #9 applied correctly here.

Patch applies with fuzz on 7.x-5.x. Works for #10 and looks good here.

 $ git apply twitter-fixed-969454-3698716_0.patch 
error: patch failed: twitter_signin/twitter_signin.module:65
error: twitter_signin/twitter_signin.module: patch does not apply
error: patch failed: twitter_signin/twitter_signin.pages.inc:14
error: twitter_signin/twitter_signin.pages.inc: patch does not apply
 $ patch -p1 < twitter-fixed-969454-3698716_0.patch 
patching file twitter_signin/twitter_signin.module
Hunk #1 succeeded at 69 with fuzz 1 (offset 4 lines).
patching file twitter_signin/twitter_signin.pages.inc
Hunk #1 succeeded at 17 with fuzz 1 (offset 3 lines).

Triggering the testbot.

Rerolled.

Committed to the 7.x-5.x and 7.x-6.x branches. Thanks.