@MelvinAguilar
Posted
Hello there ๐. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
-
In your
.box1
class, there seems to be a typo in thebackground-image
property. The "no repeat" should be moved to thebackground-repeat
property.If you want to define several properties on one line it is using
background
notbackground-image
.Corrected, it would be:.box1 { grid-row-start: 1; grid-row-end: 3; background-color: var(--moderate-violet); background-image: url(./images/bg-pattern-quotation.svg); background-repeat: no-repeat; background-position: top right; }
If you want to consolidate those background properties into a single background property, you can do it like this:
background: var(--moderate-violet) url(./images/bg-pattern-quotation.svg) no-repeat top right;
- Consider adding
max-width: 1160px
to constrain the content width on larger screens. Additionally, you can use Flexbox or Grid on the body element to center it more effectively.
I hope you find it useful! ๐
Happy coding!
Marked as helpful
@rowanDeveloper
Posted
Thanks a lot @MelvinAguilar! To be honest, your comment was really useful! I really appreciate your insight.