Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Solved some Issues #66

Closed
wants to merge 4 commits into from
Closed

Solved some Issues #66

wants to merge 4 commits into from

Conversation

azax25547
Copy link

Solved the Below

Screenshot 2020-10-08 072003
Screenshot 2020-10-08 072130
Screenshot 2020-10-08 071856
Screenshot 2020-10-08 071935

Copy link
Owner

@gouri-panda gouri-panda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@azax25547 Please don't change the style of the player.It should intact with this PR request.

@azax25547
Copy link
Author

azax25547 commented Oct 8, 2020

@azax25547 Please don't change the style of the player.It should intact with this PR request.

I just updated player names font. Anything change if you want then say, i will commit em

@gouri-panda
Copy link
Owner

@azax25547 Also Please don't replace the dark theme switch. it should be the upper right corner. Revert the change.

Aditya Narayan Mishra added 2 commits October 9, 2020 01:32
Reverting Style.css
Reverting the index.html
@azax25547
Copy link
Author

@azax25547 Also Please don't replace the dark theme switch. it should be the upper right corner. Revert the change.

Hi All the UI changes i have made are reverted back. Thanks

@NicholasACTran
Copy link
Collaborator

@azax25547 Can you fix merge conflicts then test again to double check functionality?

@NicholasACTran
Copy link
Collaborator

Closes #65
Closes #63

@azax25547 azax25547 closed this Oct 8, 2020
@azax25547 azax25547 deleted the azax branch October 8, 2020 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants