Conversation
romanoprid
left a comment
romanoprid
left a comment
There was a problem hiding this comment.
Excellent work, I think you can merge it.
redd4ford
left a comment
redd4ford
left a comment
There was a problem hiding this comment.
Good job! You might want to clean the code from comments, though.
igor-leshkevych
left a comment
igor-leshkevych
left a comment
There was a problem hiding this comment.
@JohnSteck9 code-wise everything is nice, good job. From the UI perspective, I recommend increasing top/bottom padding for "Women" label, and also adding some padding in the bottom of the screen (to increase space between bottom list items and bottom edge of the screen).
| @@ -0,0 +1,6 @@ | |||
| import 'package:flutter/material.dart'; | |||
|
|
|||
| const kTextColor = Color(0xFF535353); | |||
There was a problem hiding this comment.
I don't recommend using custom prefixes/hungarian notation/etc. in Dart. Historically, it was invented to manage complexity on the projects, but right now the direction industry moves towards is "don't create complex code, and you won't need custom prefixes for your variables)
| } No newline at end of file | ||
| } | ||
|
|
||
|
|
| ), | ||
| ]; | ||
|
|
||
| String dummyText = |
| }); | ||
| } | ||
|
|
||
| List<Product> products = [ |
4 participants
Created simple shop page
Flutter layout explorer
