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

完成”有效的学习方式”部分开发; #520

Merged
merged 1 commit into from
Jun 22, 2018

Conversation

chj-damon
Copy link

1、 用CSS实现了左侧的学习路线图;
2、将轮播图中的“开始闯关按钮”样式提取到更高级别让整个页面能共享;

@Becavalier Becavalier requested a review from S1ngS1ng June 2, 2018 05:14
Copy link
Contributor

@S1ngS1ng S1ngS1ng left a comment

Choose a reason for hiding this comment

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

改一下注释就好~没啥大问题

transform: rotate(-30deg);
}

/**有效的学习方式 开始 */
Copy link
Contributor

Choose a reason for hiding this comment

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

这里应该是 “有效的学习方式 结束” 吧

Copy link
Author

Choose a reason for hiding this comment

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

好的。

#root .study-circle .description .text {
width: 300px;
display: flex;
flex-direction: column;
Copy link
Contributor

Choose a reason for hiding this comment

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

低版本的 IE 咱们不管是吧?🌝

Copy link
Author

Choose a reason for hiding this comment

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

为整个市场放弃低版本IE做出自己的贡献!!:)

@S1ngS1ng S1ngS1ng requested a review from wudifeixue June 2, 2018 07:00
@Becavalier Becavalier merged commit 7f48663 into feature/cn-site Jun 22, 2018
<li class="gray">完成
<span class="orange">9999</span>个挑战</li>
<li class="gray">拥有
<span class="orange">9999</span>名会员</li>
</ul>
Copy link
Contributor

Choose a reason for hiding this comment

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

我就觉得,这里这么hard code真的好么。。还是说单纯为了先看下效果?

Copy link
Contributor

Choose a reason for hiding this comment

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

@wudifeixue 暂时的,place holder 而已 🌝

@wudifeixue wudifeixue deleted the feature/cn-site-chenjie branch June 24, 2018 00:47
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.

4 participants