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

fix(web): user management responsive design #5698

Merged
merged 3 commits into from
Dec 14, 2023

Conversation

martabal
Copy link
Member

Changes made in this PR

Improve the user management responsive design and add a close button on the edit user modal.

Screenshots

Before After
2023-12-14.15-47-52.mp4
2023-12-14.15-46-56.mp4

Copy link

cloudflare-workers-and-pages bot commented Dec 14, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 14f61dc
Status: ✅  Deploy successful!
Preview URL: https://acc40f88.immich.pages.dev
Branch Preview URL: https://fix-user-management-tailwind.immich.pages.dev

View logs

>
<div class="absolute inset-y-0 right-0 px-2 py-2 h-fit">
<CircleIconButton icon={mdiClose} on:click={() => dispatch('close')} />
Copy link
Contributor

Choose a reason for hiding this comment

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

While you are here, do you mind adding types for the dispatcher? In general they are missing in many places and I think it would be easy enough to add whenever we update a file that is missing them.

Copy link
Member Author

Choose a reason for hiding this comment

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

Of course

@alextran1502 alextran1502 changed the title fix: user management responsive design fix(web): user management responsive design Dec 14, 2023
@alextran1502 alextran1502 merged commit f2270ad into main Dec 14, 2023
18 checks passed
@alextran1502 alextran1502 deleted the fix/user-management-tailwind branch December 14, 2023 16:55
jonhnet pushed a commit to jonhnet/immich that referenced this pull request Dec 15, 2023
* fix: user management tailwind

* use top instead of inset-y-0

* add types to createEventDispatcher
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants