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

[fix bug 1121914] Next pager arrow wrong direction on /firefox/desktop/tips #2662

Merged
merged 2 commits into from
Jan 15, 2015

Conversation

alexgibson
Copy link
Member

  • Fixes "Next" pager arrow pointing the wrong direction.
  • Also fixes some small-screen layout issues in the header on RTL locales.

@jpetto jpetto self-assigned this Jan 15, 2015
float: left;
&:after {
content: ' »';
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Use the trailing-arrow mixin?

@jpetto
Copy link
Contributor

jpetto commented Jan 15, 2015

I'm guessing you're right about an ad blocker causing the issues in bug 1121733. Let's see if we can confirm that in today's meeting, and, if so, change the class & id on that element.

@alexgibson
Copy link
Member Author

Let's see if we can confirm that in today's meeting, and, if so, change the class & id on that element.

Sounds good to me 👍

@alexgibson alexgibson force-pushed the bug-1121914-pager-arrow-tips-page branch from 7a92cb0 to 3b032f0 Compare January 15, 2015 18:48
@alexgibson
Copy link
Member Author

@jpetto - updated with a 2nd commit that should hopefully fix bug 1121733

@jpetto
Copy link
Contributor

jpetto commented Jan 15, 2015

LGTM. Couldn't reproduce the issue from bug 1121733 (no matter how aggressive I configured ABP), but the (potential, hopeful) fix shouldn't hurt anything. r+!

jpetto added a commit that referenced this pull request Jan 15, 2015
…-page

[fix bug 1121914] Next pager arrow wrong direction on /firefox/desktop/tips
@jpetto jpetto merged commit 60632a8 into mozilla:master Jan 15, 2015
@alexgibson alexgibson deleted the bug-1121914-pager-arrow-tips-page branch January 16, 2015 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants