KendallDoesCoding / mogul-christmas

Ludwig Ahgren made a Christmas Album in 2020. I just thought of a project cause I was bored where you can play all the songs by a click of a button which you can already do from his timestamps but what makes it special is that it has lyrics of those songs too and a bunch of other features.
https://kendalldoescoding.tech/mogulchristmas
Creative Commons Zero v1.0 Universal
23 stars 62 forks source link

[BUG] Background not displaying on phone/small screen size. #259

Closed KendallDoesCoding closed 2 years ago

KendallDoesCoding commented 2 years ago

The background isn't displaying on phones or on a small screen size for some reason, for both lightmode and darkmode on the main page, entire lyrics directory, you may like page. idk what the issue is.

KendallDoesCoding commented 2 years ago

This issue has only come in too place since we made a Christmas theme background, v16.5.0.

KendallDoesCoding commented 2 years ago

@TechStudent11 Can you help figure out the issue on this?

samyberkane04 commented 2 years ago

@TechStudent11 can i work on this issue please ?

samyberkane04 commented 2 years ago

@TechStudent11 there is no background Capture d’écran de 2022-03-29 13-18-00 :

KendallDoesCoding commented 2 years ago

@TechStudent11 can i work on this issue please ?

yeah ofc.

KendallDoesCoding commented 2 years ago

@TechStudent11 there is no background Capture d’écran de 2022-03-29 13-18-00 :

There is?

image image

KendallDoesCoding commented 2 years ago

@samyberkane04 There is an issue with your end with the background, can you record a screen share video opening, https://kendalldoescoding.gq/mogulchristmas so I can try to debug the issue, the background works fine for me on both localhost and on the website. Maybe it didn't load properly or your internet had some issue, please try reloading the webpage.

KendallDoesCoding commented 2 years ago

@samyberkane04 Useful info: The background stops showing at dimensions: 621 x 666... It shows till then.

samyberkane04 commented 2 years ago

@KendallDoesCoding

that's what I did :

Capture d’écran de 2022-03-30 10-44-32

KendallDoesCoding commented 2 years ago

@KendallDoesCoding

that's what I did :

Capture d’écran de 2022-03-30 10-44-32

Well, if it works for you then no issues, but for me it doesn't render properly on mobile/small screen size, idk why. I'll figure it out.

KendallDoesCoding commented 2 years ago

@samyberkane04 The background works for 621 x 666, but stops working at 620 x 666.

samyberkane04 commented 2 years ago

@KendallDoesCoding I changed the code to get this result, when you change your window size(small screen size) the bakground don't stops working.

KendallDoesCoding commented 2 years ago

@KendallDoesCoding I changed the code to get this result, when you change your window size(small screen size) the bakground don't stops working.

Oh ok, please make a PR, if you need help feel free to ask :).

samyberkane04 commented 2 years ago

@KendallDoesCoding

I did an other PR, I think I solved this issue.

KendallDoesCoding commented 2 years ago

@KendallDoesCoding

I did an other PR, I think I solved this issue.

Thanks but the deploy preview aren't working for some reason, check my comments in #264!

samyberkane04 commented 2 years ago

@KendallDoesCoding ok I will do an other PR .

KendallDoesCoding commented 2 years ago

Thanks a lot @samyberkane04 for your help.

KendallDoesCoding commented 2 years ago

@samyberkane04 The first button is 2 lines instead of one line, please fix.

KendallDoesCoding commented 2 years ago

https://6245e49cb0b3d7000836b837--mogulchristmas.netlify.app/

KendallDoesCoding commented 2 years ago

@samyberkane04 The first button is 2 lines instead of one line, please fix.

I've reverted the changes I merged, please fix this.

samyberkane04 commented 2 years ago

@KendallDoesCoding

ok I will change it.

samyberkane04 commented 2 years ago

@KendallDoesCoding

i did the changes, but i can't always do a PR.

KendallDoesCoding commented 2 years ago

thanks

On Thu, 31 Mar 2022 at 23:26, samyberkane04 @.***> wrote:

@KendallDoesCoding https://github.com/KendallDoesCoding

i did the changes.

— Reply to this email directly, view it on GitHub https://github.com/KendallDoesCoding/mogul-christmas/issues/259#issuecomment-1084927913, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUGJUTJIVLVIJ3MH4VRQ7FDVCXRNZANCNFSM5R6ALUFA . You are receiving this because you were mentioned.Message ID: @.***>

KendallDoesCoding commented 2 years ago

@samyberkane04 Reverting your changes as I believe it messed up a lot of stuff in the directories.

samyberkane04 commented 2 years ago

@KendallDoesCoding

I think the code that I do it's works for this issue . (it's works for both modes ) I don't understand where is the problem ?

KendallDoesCoding commented 2 years ago

@KendallDoesCoding

I think the code that I do it's works for this issue . (it's works for both modes ) I don't understand where is the problem ?

Yeah but the lyrics direcotry is messed up, even lud socials.

KendallDoesCoding commented 2 years ago

I added this a long time back, that's the issue:

@media screen and (max-width: 620px) {
  body {
    background-image: none;
  }
}
KendallDoesCoding commented 2 years ago

thanks

On Thu, 31 Mar 2022 at 21:46, samyberkane04 @.***> wrote:

@KendallDoesCoding https://github.com/KendallDoesCoding ok I will do an other PR .

— Reply to this email directly, view it on GitHub https://github.com/KendallDoesCoding/mogul-christmas/issues/259#issuecomment-1084802183, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUGJUTL5RNSVPBAGYGRAFOTVCXFWHANCNFSM5R6ALUFA . You are receiving this because you were mentioned.Message ID: @.***>