Skip to content
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.

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

updating the placeholder section in the resume page #131

Merged
merged 3 commits into from
Jan 16, 2025

Conversation

ak-0283
Copy link
Contributor

@ak-0283 ak-0283 commented Jan 15, 2025

Fixes #85

Description:

As part of my web development projects, I designed and implemented dynamic input forms with an emphasis on user experience (UX). One such form included a percentage input field, where I incorporated several features:

  • Placeholder Text: The input field provides a user-friendly placeholder to guide users, ensuring clarity about the expected input format.
  • Input Validation: The field restricts input values between 0 and 100, ensuring that users can only enter valid percentage values.
  • UX Enhancements: The field is styled to match the form's overall aesthetic, and proper tooltips and error messages are displayed if the user enters an invalid value, contributing to a seamless experience.
  • Responsive Design: The form and its elements are responsive, ensuring accessibility and ease of use across various devices.

This project showcases my proficiency in creating interactive forms, focusing on data validation, UI design principles, and enhancing the overall user experience, making web applications more intuitive and efficient.

Type of PR

  • Bug fix
  • Feature enhancement
  • Documentation update
  • Other (specify): _______________

Screenshots / videos (if applicable)


Screenshot 2025-01-15 212200

Checklist:

  • I have performed a self-review of my code
  • I have read and followed the Contribution Guidelines.
  • I have tested the changes thoroughly before submitting this pull request.
  • I have provided relevant issue numbers, screenshots, and videos after making the changes.
  • I have commented my code, particularly in hard-to-understand areas.

Additional context:

  • Added a percentage input field with validation and clear placeholder text ("Enter percentage (0 - 100%)") to guide the user on expected input.
  • Ensured that input is constrained to valid percentage values (0-100) and handled error messages gracefully.
  • The changes were tested across multiple browsers and devices for responsiveness and usability.

Copy link

vercel bot commented Jan 15, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
scd-profile-score ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 16, 2025 5:38am

Copy link

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTE.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

@ak-0283
Copy link
Contributor Author

ak-0283 commented Jan 15, 2025

please review and merge it. @Ojas-Arora

@Ojas-Arora
Copy link
Owner

@ak-0283 Update Branch and Ready to merge.

@ak-0283
Copy link
Contributor Author

ak-0283 commented Jan 15, 2025

@ak-0283 Update Branch and Ready to merge.

I have updated now

@Ojas-Arora
Copy link
Owner

@ak-0283 Update Again

@ak-0283
Copy link
Contributor Author

ak-0283 commented Jan 15, 2025

you can see i have updated. @Ojas-Arora
Screenshot 2025-01-15 234244

@Ojas-Arora
Copy link
Owner

@ak-0283 Update Branch and Ready to merge.

I have updated now

Need to Update after This.

@ak-0283
Copy link
Contributor Author

ak-0283 commented Jan 15, 2025

Which branch are you telling to update

@Ojas-Arora
Copy link
Owner

@ak-0283 That one from which You have made this Pr.

@ak-0283
Copy link
Contributor Author

ak-0283 commented Jan 15, 2025

@ak-0283 That one from which You have made this Pr.

I have already attached the image and updated the branch 🙂. As you can see in the above message

@Ojas-Arora
Copy link
Owner

16.01.2025_00.14.14_REC.mp4

@ak-0283 See this there is changes in the navbar links of your Deployment in contact us and Pricing Page and This pr has been merged latest before your Pr has been made which will be rresolved by doing Branch Updation. The photo I think is the Updation I have told you firstly and that has been updated but the updation I told again has not been updated.

@ak-0283
Copy link
Contributor Author

ak-0283 commented Jan 16, 2025

now it's ok @Ojas-Arora , or else I am doing in wrong way 😔


Screenshot 2025-01-16 110959

Screenshot 2025-01-16 111009

Copy link
Owner

@Ojas-Arora Ojas-Arora left a comment

Choose a reason for hiding this comment

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

Approved

@Ojas-Arora Ojas-Arora merged commit c8bdd95 into Ojas-Arora:main Jan 16, 2025
2 checks passed
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.

Enhancing UI/UX with Placeholder Text for Improved Usability
2 participants