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
Styleguide Update #8
Conversation
Jmartin/header-menu
Merge pull request #2 from E2E-Squad/styleguide
…/ImageContainer # Conflicts: # components/Atoms/Button/Button.md # components/Atoms/Button/Button.module.scss # pages/index.tsx
color: red; | ||
@import "styles/variables"; | ||
@import "styles/atoms/buttons"; | ||
.btn { |
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.
Should use a pointer cursor
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.
import styles from './Button.module.scss'; | ||
import style from "./Button.module.scss"; | ||
|
||
interface types { |
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.
Should take an onClick event
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.
rank="6H" | ||
location="Lausanne" | ||
> | ||
<svg |
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.
Should make an Icon atom component
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.
Jmartin/imageContainer-integration
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.
LGTM
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.
LGTM
Atoms
Button
Available in 2 versions:
Image Container
Contains an image and can be shaped (
Square
,Circle
,Rectangle
) and sized (Small
,Medium
,Large
,Big
) using enum properties (Shape
andSize
. It is responsive out-of-the-box unless you choose to make itAdaptive
sized in which case it will fill the parent. It is then parent's responsibility to be responsive. The image automatically covers the container in any case.Message
Message contained in a box.
Molecules
ChatCard
Similar to
Message
excepts it also takes a pseudo which will be displayed on the left of a default image on top of the message. By default the message is a left message (.chat-card-container
) but it can also be a right message (chat-card-container--right
).Nav
It takes an array of link objects and makes it a list of links.
UserCard
Exists in 2 versions:
main-user-card
)main-user-card--full
)It displays a user with a default profile picture, a pseudo and a rating (
react-stars
). There is also a warning icon on the top right but it doesnt do anything yet nor serve any purpose. In full mode it also displays a location, a rank and a button "Notifier".Organism
Header
Displays a menu with a default app icon and default user avatar. It takes The user path and the title as well a styleClass as parameters.