Skip to content

feat(441): Add Scroll Feature to Auth #447

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

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

Conversation

Ladzsz
Copy link
Contributor

@Ladzsz Ladzsz commented Jul 9, 2025

Changes

this pr fixes issue #441
added scroll feature to auth tab
Describe the changes of your PR here

Testing

checked to ensure it can scroll
If relevant, mention your manual testing here. If possible, include screenshots.

Checklist

  • Issue has been linked to this PR
  • Code has been reviewed by person creating the PR
  • Automated tests have been written, if possible
  • Manual testing has been performed
  • Documentation has been updated, if necessary
  • Changes have been reviewed by second person

@Ladzsz
Copy link
Contributor Author

Ladzsz commented Jul 9, 2025

Hey made a new pr, the old one got closed so i made a new clean one, i updated my local version to the current main branch and just added my changes should be all good now! sorry for the hastle working with git gets big sometimes haha!

@SoulKa
Copy link
Collaborator

SoulKa commented Jul 10, 2025

looks good now from the first glance. Will review it later. Btw this is a checked checkbox in Markdown [x], [checked] will not work :)

@SoulKa SoulKa self-assigned this Jul 10, 2025
@SoulKa SoulKa linked an issue Jul 10, 2025 that may be closed by this pull request
@Ladzsz
Copy link
Contributor Author

Ladzsz commented Jul 10, 2025

Understood keep me updated!

@SoulKa
Copy link
Collaborator

SoulKa commented Jul 10, 2025

Works good :) Just one thing: can you apply the 16px padding also on the bottom? There's currently no space when scrolling all way down:
image

I think you can also apply the same padding to the headers tab

@SoulKa SoulKa changed the title added scroll feature to auth feat(441): Add Scroll Feature to Auth Jul 10, 2025
@Ladzsz
Copy link
Contributor Author

Ladzsz commented Jul 10, 2025

Alright ill add padding on both tabs

@Ladzsz
Copy link
Contributor Author

Ladzsz commented Jul 10, 2025

there i added 16 px ofor auth, headers, and param.

Comment on lines +164 to +165
<div className="pb-4">
<Table className="w-full table-auto">
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think you can simply add pb-4 to the Table instead of creating a new div here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I tried that it didnt work

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So i opted to use a new div since adding that to existing ones did nothing

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

Successfully merging this pull request may close these issues.

Make Auth Tab scrollable
2 participants