Tech-N-Science / FunwithScience

MIT License
125 stars 218 forks source link

Added matrix multiplication calculator #694

Closed Samiksha-Garg closed 2 years ago

Samiksha-Garg commented 2 years ago

Related Issue

Fixes: #685

Describe the changes you've made

Initially I resolved the issue that arose due to conflicting CSS, after merging two of my PRs. Then I added the complete functionality of matrix multiplication calculator under matrices topic. I made use of interweave library to convert string to HTML object.

Checklist:

Screenshots

Orignal :

Screenshot 2022-08-27 at 2 03 32 AM

Updated :

Screenshot 2022-08-27 at 1 47 48 AM Screenshot 2022-08-27 at 1 48 18 AM Screenshot 2022-08-27 at 1 51 11 AM Screenshot 2022-08-27 at 1 51 30 AM Screenshot 2022-08-27 at 1 52 05 AM Screenshot 2022-08-27 at 1 52 21 AM
github-actions[bot] commented 2 years ago

Our team will soon review your PR. Thanks @Samiksha-Garg :)

Amit366 commented 2 years ago

@Samiksha-Garg did u change some words in the definition or u directly copied it?

Samiksha-Garg commented 2 years ago

@Samiksha-Garg did u change some words in the definition or u directly copied it?

I had made changes!

github-actions[bot] commented 2 years ago

Thank you @Samiksha-Garg for taking out your valuable time in order to contribute to our project. Looking forward for more such amazing contributions :).

Amit366 commented 2 years ago

@Samiksha-Garg on merging there was merge conflicts so pull again and then send the pull request