-
Notifications
You must be signed in to change notification settings - Fork 2
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
Offers mockup #16
Offers mockup #16
Changes from 12 commits
6f5e96f
69d2ae5
8eb3e37
322c5ee
abce66e
b4ff597
06ac667
33a6976
8977095
a73cea4
77dac33
1842c8c
f301525
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -69,7 +69,6 @@ ul { | |
max-width: 1120px; | ||
width: 100%; | ||
margin: auto; | ||
display: flex; | ||
box-sizing: border-box; | ||
} | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,6 @@ | ||
.header { | ||
padding: 139px 0 85px 0; | ||
display: flex; | ||
.title { | ||
display: flex; | ||
flex-direction: column; | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,10 +5,11 @@ import Footer from '../Footer'; | |
import Header from '../Header'; | ||
|
||
export default function Container(props) { | ||
const header = props.close ? null : <Header />; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
return ( | ||
<div> | ||
<AppBarVG /> | ||
<Header /> | ||
{ header } | ||
{props.children} | ||
<Footer /> | ||
</div> | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,69 @@ | ||
import * as React from 'react'; | ||
import { | ||
FontIcon, | ||
Paper, | ||
} from 'material-ui'; | ||
import * as _ from 'lodash'; | ||
|
||
export default function Comments(props) { | ||
const stars = () => _.times(5, i => <FontIcon key={i} className="fa fa-star"/>); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. why not a value?
|
||
return ( | ||
<div className="comments"> | ||
<Paper style={{ width: '100%' }} zDepth={2} rounded={false}> | ||
<div className="comments-cont"> | ||
<div className="references"> | ||
55 References | ||
</div> | ||
<div className="ratings"> | ||
<div className="row"> | ||
<div className="rate"> | ||
<div className="type">Bike performance</div> | ||
<div className="stars">{stars()}</div> | ||
</div> | ||
<div className="rate"> | ||
<div className="type">Communication</div> | ||
<div className="stars">{stars()}</div> | ||
</div> | ||
</div> | ||
<div className="row"> | ||
<div className="rate"> | ||
<div className="type">Mechanical condition</div> | ||
<div className="stars">{stars()}</div> | ||
</div> | ||
<div className="rate"> | ||
<div className="type">Rating</div> | ||
<div className="stars">{stars()}</div> | ||
</div> | ||
</div> | ||
</div> | ||
<div className="comment"> | ||
<div className="author"> | ||
<div className="picture"></div> | ||
<div className="fullname">Gustaw Mikolajczyk</div> | ||
<div className="location">Warsaw, Poland</div> | ||
</div> | ||
<div className="text"> | ||
The motorcycle looks and rides as described. Mint condition, raw power. | ||
<br/> | ||
I can also recommend the owner, he is my dad. | ||
</div> | ||
</div> | ||
<div className="comment"> | ||
<div className="author"> | ||
<div className="picture"></div> | ||
<div className="fullname">Henryk Mikolajczyk</div> | ||
<div className="location">Warsaw, Poland</div> | ||
</div> | ||
<div className="text"> | ||
Aba. By that I mean a comment which is actually quite in-depth, but I can't really | ||
<br/> | ||
articulate it well just yet, because I am only two years old. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 😂 |
||
</div> | ||
</div> | ||
</div> | ||
<div className="view-all"> | ||
View all references | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. should be a link There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
</div> | ||
</Paper> | ||
</div>); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
?? what's this for?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's convenient way to keep images with different dimensions in the same frame (I assume that pictures from users have different sizes)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep, but do we use it as backgrounds? Is that better then
<img>
?