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

Add responsive for BackTop #8719

Merged
merged 1 commit into from Dec 24, 2017

Conversation

Projects
None yet
2 participants
@JetRunner
Contributor

JetRunner commented Dec 24, 2017

Problem to solve

In our project of Lintcode, we encounter such problem:
img_0141
As you can see from the screenshot, the BackTop lies on a annoying position which interfere users from viewing the content.
I fully understand that antd is not completely designed for mobile, but some small sites( like us :) ) may use responsive design to avoid heavy work for designing mobile site.
So I think it's a wonderful idea to add responsive to BackTop component.

Diff

I modify the right property for:

screen right
xs 20px
md 60px
lg+ 100px

Screenshots

MD

jietu20171224-164219 2x

XS

jietu20171224-164114 2x

@JetRunner JetRunner changed the title from WIP Add responsive for BackTop to Add responsive for BackTop Dec 24, 2017

@afc163 afc163 merged commit 603cb9e into ant-design:feature-3.1 Dec 24, 2017

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build failed
Details
security/snyk No new issues
Details

@JetRunner JetRunner deleted the JetRunner:feature-3.1 branch Dec 25, 2017

lixiaoyang1992 added a commit to lixiaoyang1992/ant-design that referenced this pull request Jan 3, 2018

lixiaoyang1992 added a commit to lixiaoyang1992/ant-design that referenced this pull request Jan 3, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment