@andreimaier
Posted
Hi there! π First off, great job on implementing the design!
Here are a few suggestions on how you can improve your code:
- try setting
display: flex
to yourbody
element
Syntax
body {
min-height: 100vh;
display: flex;
align-items: center;
justify-content: center;
}
This will align your div.center
vertically.
- add an
alt
description to yourimg.qr-code
For More info on when you should use alt
attributes and when not to, check THIS π
- try to use Semantic HTML as much as possible
- for example by replacing your
div.center
with themain
element - you can also add an
h1
heading, give it asr-only
class and hide it using something like this
- for example by replacing your
.sr-only {
position: absolute;
width: 1rem;
height: 1rem;
padding: 0;
margin: -1rem;
overflow: hidden;
white-space: nowrap;
border: 0;
}
I'm also new, but I learned a lot from comments and looking at others' solutions.
If you have any questions or need further clarification, feel free to reach out to me. I'll do my best to help!
**Keep up the good work!!**ππ»