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.

Sign up for GitHub

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

Jump to bottom

UI theme #24

Merged
merged 8 commits into from
Jun 16, 2023
Merged

UI theme #24

merged 8 commits into from
Jun 16, 2023

Conversation

pjoterini
Copy link
Collaborator

No description provided.

@vercel
Copy link

vercel bot commented Jun 13, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
guest-registration ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 16, 2023 7:12pm

@MaciejDurski
Copy link
Owner

image
zrób sticky ten navbar może trochę dziwnie wygląda ten scroll bar

@MaciejDurski
Copy link
Owner

image
jak otwiera się select to navbar skacze

@MaciejDurski
Copy link
Owner

MaciejDurski commented Jun 14, 2023 •
edited
Loading

image też masz że masz tylko dni wyświetlone? na localu w 7a3e0c2e9a1bbf6ba9c22c75bee66c55d2d77484 pokazywało całą date

@MaciejDurski
Copy link
Owner

co myślisz żeby button wyślij dać do prawej?

@MaciejDurski
Copy link
Owner

MaciejDurski commented Jun 14, 2023 •
edited
Loading

image
image

wydaje mi się że ten dolny header wygląda lepiej jak nie ma box-shadow do góry

MaciejDurski
MaciejDurski requested changes Jun 14, 2023
View reviewed changes
Copy link
Owner

@MaciejDurski MaciejDurski left a comment

Choose a reason for hiding this comment

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

najs dużo lepiej to wygląda

@MaciejDurski
Copy link
Owner

klik na logo niech odsyła do home

@pjoterini
Copy link
Collaborator Author

U mnie pokazuje całą datę zawsze

MaciejDurski
MaciejDurski requested changes Jun 15, 2023
View reviewed changes
MaciejDurski
MaciejDurski reviewed Jun 15, 2023
View reviewed changes
MaciejDurski
MaciejDurski reviewed Jun 15, 2023
View reviewed changes
MaciejDurski
MaciejDurski reviewed Jun 16, 2023
View reviewed changes
src/layouts/Layout.tsx Outdated
return (
<>
<Navbar />
{router.pathname.includes('admin') && <AdminNavbar />}
Copy link
Owner

Choose a reason for hiding this comment

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

includes(ADMIN_PANEL)

@MaciejDurski MaciejDurski merged commit 38d2131 into develop Jun 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MaciejDurski MaciejDurski MaciejDurski requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pjoterini @MaciejDurski