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 spacing on mobile for show page #5277

Open
FelixMalfait opened this issue May 3, 2024 · 4 comments
Open

Fix spacing on mobile for show page #5277

FelixMalfait opened this issue May 3, 2024 · 4 comments
Assignees

Comments

@FelixMalfait
Copy link
Member

cc @Bonapara

We should add space before the image and after the button group
Screenshot 2024-05-03 at 15 32 11

I think we should also add a border-bottom to the last element - here "Projects" (same at the border-top), as it makes it seem Timeline/Tasks/etc are part of project

@hnrykm
Copy link
Contributor

hnrykm commented May 3, 2024

@FelixMalfait Can I pick this up?

@FelixMalfait
Copy link
Member Author

Sure @hnrykm thanks!

cc @Bonapara for more clear directions

@Bonapara
Copy link
Member

Bonapara commented May 6, 2024

CleanShot 2024-05-06 at 12 13 12

  1. We should add a border-top (color: Light)
  2. & 3. We should add a padding of 64px before and after the empty state placeholder.

@FelixMalfait
Copy link
Member Author

@hnrykm let us know if we should unassign you or if you're working on this. Thanks a lot :)

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

No branches or pull requests

4 participants
@FelixMalfait @Bonapara @hnrykm and others