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

Remove language_css_alter() (RTL stylesheets) in favor of HTML 'dir' attribute. #305

Closed

Conversation

adamtothemoon
Copy link
Contributor

pr for: backdrop/backdrop-issues#207
For some reason the .rej files appear in all branches not just the one I was working on. Any idea why that would be?

@jenlampton
Copy link
Member

.rej files will appear (everywhere, as untracked files) until you delete them. Make sure you apply all the changes within first though!

.dropbutton-multiple .dropbutton li > * {
margin-left: 0.25em;
margin-right: 0;
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this is correct. This patch should be removing all the -rtl.css files, and adding that code into the non-rtl versions.

@jenlampton
Copy link
Member

Closing because this needs work. Please re-open when you have a new PR for review.

@jenlampton jenlampton closed this May 29, 2014
@adamtothemoon
Copy link
Contributor Author

pr for backdrop/backdrop-issues#207

@jenlampton
Copy link
Member

Looks like @adamtothemoon does not have permission to re-open his own pull request.

@jenlampton jenlampton reopened this Jun 3, 2014
Issue #1684806 by nod_: JSHint progress.js.
@@ -29,7 +44,7 @@ fieldset.vertical-tabs-pane > legend {
margin: 0;
}
.vertical-tabs .vertical-tabs-list a {
display: block;
display: block;gi
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like a typo here.

@quicksketch
Copy link
Member

Looks like there is a bunch of RTL sheets still unconverted in this PR. I've filed a replacement at #334 that includes all these changes plus the missing RTL stylesheets.

jenlampton pushed a commit to jenlampton/backdrop that referenced this pull request Jan 17, 2015
jenlampton pushed a commit to jenlampton/backdrop that referenced this pull request Mar 3, 2015
jenlampton pushed a commit to jenlampton/backdrop that referenced this pull request Mar 3, 2015
jenlampton pushed a commit to jenlampton/backdrop that referenced this pull request Apr 24, 2015
quicksketch added a commit that referenced this pull request May 8, 2015
Issue #305: Set fixed From and add Reply-to to improve deliverability of email.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants