search

QAZIMAAZARSHAD / Movie-Streaming-Website

A website to watch movies, anime, web series, and TV shows.
https://qazimaazarshad.github.io/Movie-Streaming-Website/
MIT License
263 stars 396 forks source link

Refactor the file storage for code optimization and readibility issue #569 #579

Closed purnasth closed 1 year ago

purnasth commented 1 year ago

Pull Request: Codebase Improvement - Centralized Files and Structured Organization

Description

This pull request addresses issue #569 , which focuses on enhancing the current state of the repository featuring a Movie Streaming Website. The existing codebase is in need of significant improvements to optimize maintainability and organization.

Changes Made

  1. Centralized CSS and JS Files: Consolidated all CSS and JS code into single, external files for efficient code management.
  2. Component-Based Structure: Created separate files (HTML, CSS, JS) for individual components such as cards, navbars, and footers. This restructuring greatly enhances the ease of maintenance.
  3. File Structure Reorganization: Introduced a structured file organization system, including a 'pages' folder to contain all HTML files, a 'css' folder for CSS files related to various components, and a 'js' folder for scripts associated with dynamic components.
  4. Image Standardization: Placed all movie poster images in a dedicated folder, ensuring image quality and resolution consistency. We removed blurry/stretched images and added high-definition posters.
  5. Code Refactoring: The entire codebase underwent a thorough refactoring process for improved logic, coding style, and overall structure. We removed unnecessary checks and optimized the code for better performance.
  6. Code Formatting: Checked and applied Prettier formatting to all pages to ensure code consistency.

Additional Notes

This was undeniably one of the most challenging and intricate tasks I've undertaken. The existing codebase was complex and required careful disentangling to achieve a more organized and efficient structure. The code was, to put it politely, one of the most intricate and messy I've encountered. However, I am confident that these improvements will significantly enhance the project's maintainability and contribute to a more streamlined codebase.

If further adjustments, refinements, or specific guidelines are needed, please provide your feedback. We are committed to delivering these enhancements effectively.

Closes #569

QAZIMAAZARSHAD commented 1 year ago

@purnasth please host your branch on GitHub pages and share the live website link, so I can review and make sure all the functionalities are still intact.

purnasth commented 1 year ago

Sorry to say but not every code is 100% efficient. While trying to fix one issue there, I had to encounter many of the others and it took more than an hour and it was a very tedicious task. So some of the bugs still can be fixed by yourself after checking it.

QAZIMAAZARSHAD commented 1 year ago

Sorry to say but not every code is 100% efficient. While trying to fix one issue there, I had to encounter many of the others and it took more than an hour and it was a very tedicious task. So some of the bugs still can be fixed by yourself after checking it.

Thank you for your efforts in attempting to refactor the codebase, I genuinely appreciate your contributions. However, it's essential to ensure that the functionality of the website remains intact as we make these improvements.

I completely understand that refactoring can be a challenging and time-consuming task, especially in a project with such an extensive codebase. That's precisely why I encourage breaking down the work into manageable sub-tasks, as described in the task description. By tackling smaller portions of the project one step at a time, we can maintain a functioning codebase throughout the process.

For instance, you could start by organizing the images for a few HTML pages or centralizing the CSS for a specific component. Like try to structure all the images in a dedicated folder for a few HTML pages, or try removing the scroll-to-top button CSS from all HTML files and creating a single CSS file for it and linking it with all pages, etc. This incremental approach will help you make steady progress without compromising the website's core functionality.

Feel free to open an issue and propose changes that align with this structured, step-by-step strategy.

Please feel free to reach out if you have any questions.

purnasth commented 1 year ago

https://www.purnashrestha.com.np/Movie-Streaming-Website

QAZIMAAZARSHAD commented 1 year ago

https://www.purnashrestha.com.np/Movie-Streaming-Website

@purnasth please share a link that is based on your branch. This link doesn't have the changes you made.

You can use GitHub pages for your branch.