Skip to content

Frontend Fixes - #343

Merged
Swe-yy merged 26 commits into
devfrom
patch/ui-Taskeen
Sep 16, 2026
Merged

Swe-yy merged 26 commits into
devfrom
patch/ui-Taskeen

Conversation

@TaskeenAbdoola

@TaskeenAbdoola TaskeenAbdoola commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

Merge frontend fixes for:

  1. Popup
  2. Match searching
  3. Match found
  4. Country codes for sign up
  5. Clearer loading states for sign in and sign up.
  6. Cleaner UI for forgot password.
  7. Removed bg for sign in and sign up, i will fix the form to not be so light when i do themes later on.

Related issue

How to test

cd frontend
npm install country-flag-icons

@codecov

codecov Bot commented Sep 14, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
frontend/src/Views/SignUp.tsx 0.00% 3 Missing ⚠️
frontend/src/Views/ForgotPassword.tsx 0.00% 2 Missing ⚠️
frontend/src/Views/MatchSearching.tsx 0.00% 1 Missing ⚠️
frontend/src/Views/SignIn.tsx 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@Swe-yy Swe-yy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm still not seeing loading states on the sign in, sign up and log out buttons.
i'm only seeing it on the confirmation code page

Comment thread frontend/src/Views/SignIn.tsx Outdated
Comment thread frontend/src/Views/Popup.tsx
Comment thread frontend/src/Views/SignUp.tsx
@nsmzoneli

Copy link
Copy Markdown
Contributor
image Maybe increase the width of the buttons ? They seem a bit skinny

@nsmzoneli

Copy link
Copy Markdown
Contributor
image

Typo in the "Create an account" button 🙏

@sonarqubecloud

Copy link
Copy Markdown

@nsmzoneli nsmzoneli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

all the UI changes that the PR seeks to merge in are there and working.

Image

@Swe-yy Swe-yy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The buttons have visible loading states now and the styling updates look great

@Swe-yy
Swe-yy merged commit 0939f08 into dev Sep 16, 2026
6 of 7 checks passed
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.

3 participants