Feature: Calculate String Function Implementation #3

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.

Sign up for GitHub

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

Jump to bottom

Open

rahul-p-rajesh wants to merge 7 commits into incubyte:main from rahul-p-rajesh:main

rahul-p-rajesh commented Oct 14, 2025

What did you do?

Please include a summary of the changes.

Added accessibility fixes to the calculator form
Replaced non-semantic button with a semantic <button> element
Added proper <label> elements for all inputs and linked them via htmlFor
Added descriptive alt text to the banner image
Improved color contrast for better readability
Used aria-required, aria-invalid, and aria-describedby for form fields
Ensured error and result messages are accessible
Updated form to use onSubmit for better keyboard accessibility

rahul-p-rajesh and others added 7 commits

October 14, 2025 20:50


          feat: string calculator base logic

605d3e3


          feat: handle edge case in string calculator

3e43b7e


          feat: add string calculator logic to input

225cba5


          fix: accessibilit fixes

e9ec6cd


          Merge pull request #1 from rahul-p-rajesh/feature/string-calculator

a61c2f7

Feature/string calculator


          add unit test cases

cb76f6b


          Merge pull request #2 from rahul-p-rajesh/feature/string-calculator

b06dd00

add unit test cases

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

Labels

None yet