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

Missing Validation in Newsletter , SignUp Page, and Login Page | Text Justification | BackToTop Scrolling Issue and ....... #444

Closed
Shariq2003 opened this issue Oct 8, 2024 · 7 comments

Comments

@Shariq2003
Copy link
Contributor

On the website, multiple fixes are required like :

  1. Newsletter Email validation.
  2. Newsletter Success/Failure Popup Alert Missing.
  3. BackToTop Button Hard scroll to the top without smooth scrolling.
  4. Text Justification is not there in home page cards.
    Also, refer to this video.
bandicam.2024-10-08.21-19-57-755.mp4

Please assign this to me.

Copy link

github-actions bot commented Oct 8, 2024

🙌 Thank you for bringing this issue to our attention! We appreciate your input and will investigate it as soon as possible !

@Shariq2003 Shariq2003 changed the title Missing Validation in Newsletter , SignUp Page, and Login Page | Text Justification | BackToTop Scrolling Issue Missing Validation in Newsletter , SignUp Page, and Login Page | Text Justification | BackToTop Scrolling Issue and ....... Oct 8, 2024
@mansiruhil13
Copy link
Owner

mansiruhil13 commented Oct 8, 2024

you can work on this issue but there is issue in navbar section that's assigned already, you have to make changes according to that
Screenshot 2024-10-08 225227
Screenshot 2024-10-08 230037
Screenshot 2024-10-08 230212

@Shariq2003
Copy link
Contributor Author

Shariq2003 commented Oct 8, 2024

you can work on this issue but there is issue in navbar section that's assigned already, you have to make changes according to that Screenshot 2024-10-08 225227 Screenshot 2024-10-08 230037 Screenshot 2024-10-08 230212

okay @mansiruhil13 mam

@mansiruhil13
Copy link
Owner

actually whole home page broken assigned already, make changes for newsletter, signup .. according to that. so that when other pr merged, this issue will not create any other issue again.

@Shariq2003
Copy link
Contributor Author

actually whole home page broken assigned already, make changes for newsletter, signup .. according to that. so that when other pr merged, this issue will not create any other issue again.

okay @mansiruhil13

@Shariq2003
Copy link
Contributor Author

@mansiruhil13 mam, please review again and merge my PR

MastanSayyad added a commit that referenced this issue Oct 28, 2024
Added Newsletter Valiadation | Toast Alerts | Smoth Scrolling | Issue #444
Copy link

✅ This issue has been closed. Thank you for your contribution! If you have any further questions or issues, feel free to raise them, and we can discuss more!

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

No branches or pull requests

2 participants