QR Code using CSS flexbox

Community feedback

• 

  devkhrmnturk•60

  @devkhrmnturkPosted about 2 months ago

  I think it is very good that you are using the REM values. You may be converting these values ​​from px. If you do so, congratulations. it is not too far from the design in general (instead of giving the padding feature in every direction, you can give it in more detail, for example: padding: 16px 16px 40px 16px; clockwise, according to the features of the given design It might make sense to implement it this way. The border-radius you use corresponds to 10px, what I see in the design is 20px (1.25 rem)) I think these are the reasons for the slight difference from the general design, the structure you have established is simple, plain and understandable. I didn't have much difficulty understanding it. This is quite nice. I think you can give the class names in a little more detail, for example (text-content / text-contents), I think it will be more understandable, seeing similar names can be confusing.

  0