Skip to content

Conversation

biswajit760
Copy link
Contributor

@biswajit760 biswajit760 commented Sep 24, 2025

Description

This PR restructures the project to follow standard web development practices by separating HTML, CSS, and JavaScript files. Previously, all styling and scripts were embedded directly within HTML files, which made maintenance and scalability difficult.

Changes Made:

  • Moved inline CSS from HTML files to separate .css files for better readability and maintainability.

  • Moved inline JavaScript from HTML files to separate .js files for modularity and cleaner code structure.

  • Updated HTML files to correctly link the new CSS and JS files.

  • Ensured that all pages maintain their original functionality and styling after the separation.

#600 Issue No

Benefits:

  • Improved code readability and organization.

  • Easier maintenance and future enhancements.

  • Better adherence to web development best practices.

Testing:

  • Verified that all pages render correctly.

  • Tested all JavaScript functionalities to ensure no breakage.

  • Checked responsiveness and styling consistency across different pages.

Type of change

  • Bug fix
  • New feature
  • Improvement
  • Documentation update

How Has This Been Tested?

Explain how you tested your changes.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code where necessary
  • I have added tests that prove my fix is effective or that my feature works
  • I have made corresponding changes to the documentation

@github-actions
Copy link

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@supriya46788 supriya46788 merged commit 29af595 into supriya46788:main Sep 25, 2025
1 check 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.

2 participants