modernxmpp / easy-xmpp-invitation

Easy XMPP Invitation Landing Page
MIT License
41 stars 20 forks source link

QR image to fit page width (CSS) #36

Closed sjehuda closed 1 year ago

sjehuda commented 2 years ago

image

ge0rg commented 2 years ago

The QR code is meant for desktop clients, so that you can scan it with your mobile device from the desktop screen. There is no use in showing the QR code on a mobile screen, as you are the recipient of the invitation and not the sender.

sjehuda commented 2 years ago

~I didn't know that.~ I understand, now. Thank youfor the explanation. I hope I'll take care of this width issue soon.

sjehuda commented 2 years ago

This centers the image.

-    <div class="qrcode" id="qrcode"></div>
+    <div class="qrcode img-center" id="qrcode"></div>

What exactly the following code achieves?

.qrcode {
    padding: 20px 5px 5px 5px;
    width: 42.5%;
    height: 100%;
    margin: auto;
}

I don't want to change the CSS code without asking.

sjehuda commented 1 year ago

Removing width: 42.5%; seems to solve this issue.

However, it seems that we need to scale it when screen width is more narrow.