Skip to content

Task practice-react-styling done#71

Open
satoshi300 wants to merge 7 commits intodevmentor-pl:masterfrom
satoshi300:master
Open

Task practice-react-styling done#71
satoshi300 wants to merge 7 commits intodevmentor-pl:masterfrom
satoshi300:master

Conversation

@satoshi300
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown
Owner

@devmentor-pl devmentor-pl left a comment

Choose a reason for hiding this comment

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

Michale,

Całość jest ok! 👍
Zostawiłem 2 komentarze - warto wziąć pod uwagę w przyszłości ;)

</Col>
</Row>

<ThemeProvider theme={theme}>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Raczej wrzuciłbym tylko dla 2 kolumny (bo tam używamy styledComponents)

{/* <Button active>Active</Button> */}

{/* <Button disabled>Disabled</Button> */}
</ThemeProvider>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

👍

Library
</Breadcrumb.Item>
<Breadcrumb.Item active>Data</Breadcrumb.Item>
</Breadcrumb>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

👍

<h3>Contact</h3>
<p>This is contact content.</p>
</Tab>
</Tabs>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

👍


<Button>Go somewhere</Button>
</Card.Body>
</Card>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

👍

pointer-events: none;
}
`}
`; No newline at end of file
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

👍

);
};

export default BreadcrumbItem; No newline at end of file
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

👍

);
}

export default Button; No newline at end of file
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

👍

`}
`;

export { StyledButton }; No newline at end of file
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

👍

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Trochę wygląda inaczej niż w przykładzie. Na rekrutacji może to być ocenione jako błąd.

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.

2 participants