telescope icon indicating copy to clipboard operation
telescope copied to clipboard

Sign-up flow confirmation page has overflow

Open AmasiaNalbandian opened this issue 2 years ago • 6 comments

What happened: There is overflow in the confirmation page for the sign-up flow chrome_Xcc4I4UEmW

What should have happened: Remove the overflow or handle it.

How to reproduce it (as precise as possible): Complete sign up link and get to the confirmation page.

AmasiaNalbandian avatar May 17 '22 04:05 AmasiaNalbandian

Shall i work on this issue?

cypersii avatar May 17 '22 08:05 cypersii

By all means, @cypersii.

humphd avatar May 17 '22 16:05 humphd

To make this issue hacktoberfest ready, the following needs to be improved:

  • Where is this page/issue located? (link on the website; location in the code)
  • ~~Clean up issue text~~

sirinoks avatar Jul 25 '22 00:07 sirinoks

To make this issue hacktoberfest ready, the following needs to be improved:

  • Where is this page/issue located? (link on the website; location in the code)
  • ~Clean up issue text~

The file to change can be found here: https://github.com/Seneca-CDOT/telescope/blob/master/src/web/app/src/components/SignUp/Forms/Review.tsx

AmasiaNalbandian avatar Jul 25 '22 15:07 AmasiaNalbandian

Hi, I would like to take this issue for the hacktoberfest, can I take it?

DiegoHdz7 avatar Sep 28 '22 17:09 DiegoHdz7

@DiegoHdz7 sure, go for it.

humphd avatar Sep 28 '22 17:09 humphd

Hello, it seems no one is working on this issue. May I take it?

cychu42 avatar Oct 24 '22 17:10 cychu42

@cychu42 all yours!

manekenpix avatar Oct 24 '22 17:10 manekenpix

I managed to find that the long RSS URL at the bottom is the culprit, so I added word-break: 'break-word' to the CSS for blogRSS class div, which solves the overflow issue presented.

However, I found another issue while testing. I'm not sure how to make the container class div(the one with blue background) auto adjust in case the user wants to add more than 1 of those long RSS. As you can see in the picture below, a second Twitch URL would overlap with the buttons. review_page

I can just set the height to bigger...but I doubt it’s a good solution.

cychu42 avatar Oct 25 '22 19:10 cychu42

cc @PedroFonsecaDEV. Maybe we should use trailing ellipses ... to shorten it, and show the whole thing on hover?

humphd avatar Oct 25 '22 21:10 humphd

My first thought was making a scrollable box, but the ellipses and hover idea would look way better. I can try to implement the idea and make a PR for further review.

cychu42 avatar Oct 27 '22 12:10 cychu42