Skip to content
This repository was archived by the owner on Dec 19, 2024. It is now read-only.

Fixed display element on small device #138

Merged
merged 1 commit into from
May 29, 2015
Merged

Fixed display element on small device #138

merged 1 commit into from
May 29, 2015

Conversation

magsout
Copy link
Contributor

@magsout magsout commented May 29, 2015

  • Position Header element on small device
    header
  • Position features on small device
    features
  • LIne height list of tools on small device

usage

@MoOx
Copy link
Owner

MoOx commented May 29, 2015

Can you provide screenshots before/after please ?

@@ -46,67 +46,67 @@ title: cssnext
<div class="r-Grid-cell r-minM--2of3">
<h2 class="cssnext-Section-title"><a href="/features/">Features</a></h2>
<ul class="r-Grid cssnext-List cssnext-FeaturesList">
<li class="r-Grid-cell r-all--1of2">
<li class="r-Grid-cell r-all--1of2 r-maxS--2of2">
Copy link
Owner

Choose a reason for hiding this comment

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

Hum, 2of2 seems to be 100%, which is the default value right ? Maybe we should replace r-all--1of2 by r-minL--1of2 instead (or minM) ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

.r-minS--1of2,is better, no ?

Copy link
Owner

Choose a reason for hiding this comment

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

probably :D

@magsout
Copy link
Contributor Author

magsout commented May 29, 2015

Done

@MoOx
Copy link
Owner

MoOx commented May 29, 2015

w00t.

MoOx added a commit that referenced this pull request May 29, 2015
Fixed display element on small device
@MoOx MoOx merged commit 3d4fa07 into master May 29, 2015
@MoOx MoOx deleted the patch-website branch May 29, 2015 23:08
@MoOx
Copy link
Owner

MoOx commented May 29, 2015

Patchs online in a minute :)

MoOx added a commit that referenced this pull request May 29, 2015
@Macxim
Copy link
Contributor

Macxim commented May 29, 2015

Way better 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants