ContriHUB / MovieScreen

0 stars 13 forks source link

Frontend design modified #7 #14

Closed kirtisrivastava22 closed 1 week ago

kirtisrivastava22 commented 1 week ago
Front end of website modified ## Description Changes majorily done in stylesheet and html files Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change. ## Related Issue

Fixes # (issue) Improve Frontend Design

Type of change

Motivation and Context

Better Frontend ## How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Screenshots (if appropriate):

Screenshot 2024-10-10 155902

Checklist:

Test Configuration:

eleensmathew commented 1 week ago

Can you send screenshots of the new design? Also pls pull the recent changes and resolve the conflicts

kirtisrivastava22 commented 1 week ago

Screenshot 2024-10-10 155902 Screenshot 2024-10-10 194400

kirtisrivastava22 commented 1 week ago

Pull the recent changes means to fork again or to commit my recent changes?

eleensmathew commented 1 week ago

Pull the commits from my main branch. Also there are some html changes that were made in the screenshots that you added. More pages like movie list and show list have you modified? Also I'll give you bonus for responsiveness.

kirtisrivastava22 commented 1 week ago

Yes MovieList and showlist page also modified ...But the recent changes are yet to see

eleensmathew commented 1 week ago

Yes MovieList and showlist page also modified ...But the recent changes are yet to see

Recent changed yet to see means? Pls attach screenshots using an example movie and show? The api is omdb.

kirtisrivastava22 commented 1 week ago

Recent commit changes in html files which were not earlier present when I forked

On Thu, 10 Oct 2024, 8:14 pm eleensmathew, @.***> wrote:

Yes MovieList and showlist page also modified ...But the recent changes are yet to see

Recent changed yet to see means? Pls attach screenshots using an example movie and show? The api is omdb.

— Reply to this email directly, view it on GitHub https://github.com/ContriHUB/MovieScreen/pull/14#issuecomment-2405308352, or unsubscribe https://github.com/notifications/unsubscribe-auth/BDDSTPINVLZWSELTK5IAV2TZ22HDXAVCNFSM6AAAAABPWOWVG2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBVGMYDQMZVGI . You are receiving this because you authored the thread.Message ID: @.***>

kirtisrivastava22 commented 1 week ago

Pull Request done again https://github.com/ContriHUB/MovieScreen/pull/16#issue-2579392004 @eleensmathew

eleensmathew commented 1 week ago

Add screenshots after you have added one movie and one show pls

kirtisrivastava22 commented 1 week ago

Screenshot 2024-10-10 221258 Screenshot 2024-10-10 234015 Screenshot 2024-10-10 235252

kirtisrivastava22 commented 1 week ago

With Reference to PR :- https://github.com/ContriHUB/MovieScreen/pull/16#issue-2579392004

eleensmathew commented 1 week ago

Great i review it soon

kirtisrivastava22 commented 1 week ago

Are any more new modifications needed in the frontend design?

eleensmathew commented 1 week ago

Hey no need for more changes. I'll check it out locally and i will accept your pr.