Fix Bug 972231 - Fix the newsletter form RTL layout #1693

Merged
merged 1 commit into from Mar 6, 2014

Projects

None yet

3 participants

@kyoshino
Member

Only changes to CSS.

@sgarrity sgarrity and 1 other commented on an outdated diff Feb 28, 2014
media/css/plugincheck/plugincheck.less
- }
- .field {
- padding: 4px 0;
- }
- h3,
- .form-contents,
- .form-submit {
- width: (@gridColumnWidth * 4) + (@gridGutterWidth * 3);
- float: left;
- margin: 0 (@gridGutterWidth / 2);
- }
- h3,
- .form-submit {
- padding-top: 4px;
- }
-}
@sgarrity
sgarrity Feb 28, 2014 Contributor

Is this block just not necessary anymore?

@kyoshino
kyoshino Mar 4, 2014 Member

plugincheck.less imports sandstone-resp.less so those are just redundant. I just fixed the padding of #footer-email-form.

@craigcook
Member

r+ 🍭

@craigcook craigcook merged commit 52cc105 into mozilla:master Mar 6, 2014

1 check passed

default Jenkins build 'bedrock_github' #3385 has succeeded
Details
@kyoshino kyoshino deleted the kyoshino:bug-972231-newsletter-rtl branch Mar 26, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment