QR Code Component challenge

Although not a complicated challenge, I did do a lot of research into the best practices for laying out a single component so it fills the entire viewport without causing scrolling. My reason for this was primarily to find a resolution for the vh unit issue within Safari on mobile (footer being hidden by the menu bar). I tried many different things all of which are represented in my commits. I'd love to hear any feedback on this as I was not able to come up with a true solution. In addition, I adjusted the design to accommodate for landscape orientation and would appreciate any insight on the code I wrote for this layout. Thanks so much!!!