Refactor activity.js and activity-works.js to Remove Redundant Code and Improve Code Quality

[ravjot07]

Overview
This pull request refactors the activity.js and activity-works.js file to eliminate redundant code, enhance readability, and improve overall code quality. By organizing the code into a DrawingApp class and removing unnecessary repetitions, I made the codebase more maintainable and accessible for current and future contributor

Changes Made:-

• Removed Redundant Code by Consolidating similar Functions
• Wrapped related variables and functions within the DrawingApp class to encapsulate the this activities state and behaviour. Improved the modularity of the code.
• Simplified the way event handlers are attached to objects, reducing complexity. And Ensured consistent use of this by leveraging closures and binding methods where appropriate.
• Applied consistent code formatting for better readability.

[ravjot07]

@chimosky As suggested by you i have opened new PR for refactoring can you review and test this PR......

[quozl]

• commit message does not follow our conventions; https://github.com/sugarlabs/sugar-docs/blob/master/src/contributing.md#making-commits

Your changes would have been better done in more than one commit, separating formatting changes from refactoring changes. It would have made it easier to review.

[ravjot07]

@quozl i have changes commit message to a better suitable one. For review you can just test all the features i have created a drawing app class to encapsulate all the working and removed duplicate code from activity-works.js. like most of the code was redundant there for some reason...

[quozl]

https://github.com/sugarlabs/sugar-docs/blob/master/src/contributing.md#guide-for-reviewers is our guide for reviewers.

[ravjot07]

https://github.com/sugarlabs/sugar-docs/blob/master/src/contributing.md#guide-for-reviewers is our guide for reviewers.

Yes sir, for sure 😀