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

Join Group Screen #3

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Join Group Screen #3

wants to merge 2 commits into from

Conversation

paulinapp1
Copy link

screen which allows you to join or create a group, with input code

@paulinapp1 paulinapp1 added the ready to review PR is ready to be reviewed label Sep 11, 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.

po rebase sprawdz swoje pliki czy nie trzeba poprawić czegoś prettierem

return (
<Wrapper>
<BackButton>
<ArrowBackIcon />
Copy link
Contributor

Choose a reason for hiding this comment

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

back button jest nie w poprawnym miejscu
image

<Typography variant="h5" gutterBottom>
Podaj kod grupy, do której chcesz dołączyć
</Typography>
<GroupCodeInput>
Copy link
Contributor

Choose a reason for hiding this comment

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

podczas wpisywania kodu powinno automatycznie przy wpisaniu przenieść do następnego pola.
można na przykład użyć onChange


export const JoinButton = styled(Button)`
background-color: ${({ theme }) => theme.colors.button_red} !important;
color: white;
Copy link
Contributor

Choose a reason for hiding this comment

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

na nowym developie jest wprowadzony nowy plik theme z niego trzeba importować wszystkie kolory nawet white. Robiąc rebase zostaw swoje kolory bo każdy może tam dodawać nowe w razie potrzeby.

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