Fix alignment of courseCard

samad-yar-khan commented 3 years ago

ISSUE #247

fixes #247

Describe the changes you've made

I have added style for the 'li' component inside the useStyles object which can be found in - 'edu-client/src/pages/CoursePageViewWithVideo/index.js'.
The 'li' has been made flex and all the components inside the 'li' are now center-aligned.
The style of the 'li' are being used by the ListItem component which contains our CourseCards.
As a result the CourseCards are now center aligned for all screen sizes.

Describe if there is any unusual behavior (Any Warning) of your code(Write NA if there isn't) NA

Additional context

Here are the images before and after the changes for reference.

Before the Changes Capture2

After Changes Capture

Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] The title of my pull request is a short description of the requested changes.

samad-yar-khan commented 3 years ago

Hey @Abhishek-kumar09 , can you please review my pull request .

Abhishek-kumar09 commented 3 years ago

Looks good! Thanks for your PR @samad-yar-khan

samad-yar-khan commented 3 years ago

@Abhishek-kumar09 Thanks a lot for letting me work on the issue . First PR ever :)

codeforcauseorg / edu-client