Skip to content

Fix Fibonacci input visibility and validation issues(#489)#527

Merged
steam-bell-92 merged 1 commit into
steam-bell-92:mainfrom
siri-004:fix/fibonacci-input-validation-ui
May 21, 2026
Merged

Fix Fibonacci input visibility and validation issues(#489)#527
steam-bell-92 merged 1 commit into
steam-bell-92:mainfrom
siri-004:fix/fibonacci-input-validation-ui

Conversation

@siri-004
Copy link
Copy Markdown
Contributor

📝 Description

This PR fixes usability and validation issues in the Fibonacci Series generator modal

✅ Changes Made

  • Fixed invisible input text issue in light mode by improving input field styling and contrast.
  • Added proper validation for empty and invalid inputs.
  • Prevented Fibonacci sequence generation when the input is empty or less than or equal to 0.
  • Added user-friendly validation error messages.
  • Removed unintended default Fibonacci generation behavior.

🔗 Linked Issue

Closes #489
image


📋 Contribution Checklist

  • My code strictly adheres to the project guidelines (Standard library only, procedural code, clean spacing).
  • I have verified that my files are placed in the correct directory.
  • I have tested my changes thoroughly on my local machine.
  • I have included interactive emojis and clean console/UI outputs.
  • GSSoC 2026: I have been formally assigned to this issue and noted it above.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 21, 2026

@siri-004 is attempting to deploy a commit to the Anuj's projects Team on Vercel.

A member of the Team first needs to authorize it.

@steam-bell-92 steam-bell-92 merged commit ea59d5d into steam-bell-92:main May 21, 2026
3 of 4 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

🎉 Thank you for your contribution!

Your Pull Request has been merged successfully.

We appreciate the time and effort you put into improving this project. Contributions like yours help the repository grow and stay useful for everyone.

If you'd like to contribute again, please check the open issues and make sure you are assigned before opening another Pull Request.

Thanks again for your support! 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐛 [Bug]: Fibonacci- Input field text visibility and validation improvements.

2 participants