Skip to content

Comments

Fixes form error + hint when friend request sent#284

Open
Rgoplay wants to merge 2 commits intoMathMan05:mainfrom
Rgoplay:username-friend-request-form-fix
Open

Fixes form error + hint when friend request sent#284
Rgoplay wants to merge 2 commits intoMathMan05:mainfrom
Rgoplay:username-friend-request-form-fix

Conversation

@Rgoplay
Copy link
Contributor

@Rgoplay Rgoplay commented Feb 17, 2026

Description

Changed the overflow of form to be visible so that when an error toast is created it is shown and not hidden below with a scrollbar because of overflowing. Might not be the best solution as if the form goes outside the screen there is no scrolling.
Added a placeholder text when the friend request is sent.

Related issues

Issue #240

Added hint when friend request is sent
@Rgoplay Rgoplay marked this pull request as draft February 17, 2026 11:52
@Rgoplay
Copy link
Contributor Author

Rgoplay commented Feb 17, 2026

Reverting back overflow change because it breaks scrolling when necessary. I made error toast move the element below them which is not ideal but working.
We should probably rework the way error toast are shown.

@Rgoplay Rgoplay marked this pull request as ready for review February 17, 2026 16:36
background: var(--channel-hover);
border-radius: 0.1in;
position: absolute;
position: relative;
Copy link
Owner

Choose a reason for hiding this comment

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

why is it being switched from absolute to relative?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because when absolute, there is no space created for the error message, and it create scrolling (as the area is too small contain the error message). So the message become "hidden" behind the scroll. When relative, a space is created so elements below are moved but it doesn't create a scrollable area.

Copy link
Owner

Choose a reason for hiding this comment

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

won't that make layout shifts then?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes unfortunately, but I think that's better than just not seeing the error.
I don't really know how to make it better

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.

2 participants