Skip to content
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

feat: Manage users component #4

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

hamburger231
Copy link

Added a screen for managing users.

@hamburger231 hamburger231 self-assigned this Jul 20, 2024
@hamburger231 hamburger231 added the ready to review PR is ready to be reviewed label Jul 20, 2024
Copy link
Contributor

@Aghon5304 Aghon5304 left a comment

Choose a reason for hiding this comment

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

powiększ czcionkę "członkowie Nazwa grupy" i zwiększ odstęp między członkami grupy np gap by członkowie mieli stałą odległość między sobą oraz dodaj padding na całej stronie by było bliższe figmie. Zmień usuwanie członka na ikonę kosza z figmy
github : image

figma: image

}
`;

export const BackButton = styled(IconButton)`\
Copy link
Contributor

Choose a reason for hiding this comment

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

\ i tab do usunięcia. Po rebasie na nowego developa prettier powinien to naprawić, sprawdź od razu inne pliki.

return (
<>
<CssBaseline />
<ManageUsers />
Copy link
Contributor

Choose a reason for hiding this comment

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

ManageUsers do usunięcia z app.tsx

@@ -0,0 +1,57 @@
import * as React from 'react';
Copy link
Contributor

Choose a reason for hiding this comment

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

moduły powinny być umieszczone w src/screens/{nazwa modułu} będziesz musiał przenieść manageUseres

border-radius: 9999px;
border: 2px solid #bbbbbb;
@media {
color: black;
Copy link
Contributor

Choose a reason for hiding this comment

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

każdy kolor musi być importowane z pliku theme. Możesz go znaleźć na nowym developie. w razie potrzeby możesz do niego dopisać nowe kolory

import GroupRemoveIcon from '@mui/icons-material/GroupRemove';
import { BackButton, Item, RemoveButton, Wrapper } from './manageUsersStyle';

export const ManageUsers = () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

exportuj na końcu pliku przy użyciu export default ManageUsers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to review PR is ready to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants