Conversation
There was a problem hiding this comment.
Pull request overview
This PR adjusts the AuthCardLayout.vue layout spacing around the Footer component to create more visual separation on auth pages.
Changes:
- Increased the top spacing before the
Footer(mt-6→mt-16). - Added additional padding via a new
pt-35class on theFooter.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <slot name="outside" /> | ||
| </div> | ||
| <Footer class="mt-6 w-full" /> | ||
| <Footer class="mt-16 w-full pt-35" /> |
There was a problem hiding this comment.
PR description mentions increasing spacing above and below the Footer, but this change only increases top margin (mt-16) and top padding (pt-*). If bottom spacing is desired, consider adding bottom margin/padding (e.g. mb-* or py-*) rather than (or in addition to) pt-*.
| <slot name="outside" /> | ||
| </div> | ||
| <Footer class="mt-6 w-full" /> | ||
| <Footer class="mt-16 w-full pt-35" /> |
There was a problem hiding this comment.
pt-35 is not a standard Tailwind spacing utility, and there’s no Tailwind config in the repo defining a custom 35 spacing step. This class will likely be ignored/purged, so the intended padding won’t apply. Use a valid spacing token (e.g. pt-32/pt-36) or an arbitrary value like pt-[35px] instead.
| <Footer class="mt-16 w-full pt-35" /> | |
| <Footer class="mt-16 w-full pt-[35px]" /> |
This pull request makes a small styling adjustment to the
AuthCardLayout.vuelayout. The spacing above and below theFootercomponent has been increased for improved visual separation.