The autocomplete throbber in Seven theme is on the right side of input text fields. It should be on the left.

Patch attached with minor comment cleanup too.

CommentFileSizeAuthor
#3 throbber-rtl.patch609 bytesreglogge
#3 throbber.gif2.33 KBreglogge
seven-throbber.patch753 bytesreglogge
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

reglogge’s picture

Status: Active » Needs review

bot

Jeff Burnz’s picture

Can we roll this in with the big patch for Seven RTL #766458: Seven theme lacks rtl styling? Seems like it should be part of that issue.

reglogge’s picture

Status: Needs review » Postponed
Issue tags: +RTL
FileSize
2.33 KB
609 bytes

@Jeff: sure, I will try this. In the meantime however, just to get this in here, this new patch fixes the problem properly (I think...), by removing throbber styling from Seven completely. There already are proper throbber styles in system.base.css system.base-rtl.css, to which the existing styles in Seven's style.css add nothing of value. So out they go...

Also attaching a new throbber.gif which also has 2px of empty transparent space to the left of the throbber (not only to the right) for giving it a small visual padding inside its input element when rendered in RTL context.

Setting to postponed while trying to reroll #766458: Seven theme lacks rtl styling including these changes.

mgifford’s picture

Issue summary: View changes
Status: Postponed » Active

Don't think there's any reason to postpone this any more.