@whunter413
Submitted
Feedback welcome! I couldn't get the css to load on GitPages. Any advice for that? Thanks.
@FlaviuD94
@whunter413
Submitted
Feedback welcome! I couldn't get the css to load on GitPages. Any advice for that? Thanks.
@FlaviuD94
Posted
Hi @whunter413,
You could try to place the css file in the same folder as index.html, I think in this way it will run. After you did this, make a commit from your vs code then you need to go to github page change the branch to none, then save. And then you have to deploy again and in few minutes it will be back on updated. Don't forget to modify the src path for linking css file.
Have a good day and enjoy coding!
Marked as helpful
@DaftEndev
Submitted
@FlaviuD94
Posted
Hi @DaftEndev, you did a nice work on this project😀. I have some suggetions for you to improve your code: 📝when using img tag don't forget to add alt text. 📝try to use section tag online for sections of your code, like this project could be integrate in a section, and inside use div for block elements and span for inline elements. 📝you can eliminate the code that is not related to your project.
Keep up the good work and enjoy coding!
Marked as helpful
@MohLutfiFauzi
Submitted
The most difficult thing in this task is to ensure the distance between the components because we are not given instructions for it
@FlaviuD94
Posted
Hi, nice to have you around! Welcome to Frontend Mentor Coding community Your solution looks great, you can still do some things to it:
I hope it will help you, Enjoy coding!
Marked as helpful
@tdipowervag
Submitted
@FlaviuD94
Posted
Hello @tdipowervag,
Welcome to Frontend Mentor community!
Your solution looks awesome and I suggest you add one landmark to your page:
You can use the <main> instead of using <div> for the container element.
More information about Semantic HTML5 Elements I hope this helps and good luck!
Have a nice day and enjoy coding!
@FlaviuD94
Submitted
@FlaviuD94
Posted
I sow that you link is not working. Already made the change that you mention, thank you again. Only one thing I want to mention. The break point to change the images is better at 961px, if you keep it at 650, when you use a phone and turn your phone to landscape screen the style will break.
Wish you all the best!
@movileanuv
Submitted
@FlaviuD94
Posted
Hello,
For errors I used a paragraph below the inputs with position: absolute (in this way the content do not moves when it's displayed) and display:none to hide it, after that you can use an IF statemant with a condition, for exemple (cardholdername.length < 10) and change the css display if it's true. In this way you don't need to worry about the margin topic. Take a look at mine solution for this challange. https://www.frontendmentor.io/solutions/clean-css-and-vanila-javascript-rkqzzWVs9t
Have a nice day and enjoy!
@FlaviuD94
Submitted
@FlaviuD94
Posted
Hello,
Thank you for your warm welcoming! I have applied all your suggestions and I much appriciated your feedback. Also I modified h1 font-size to better match with the exemple provided in the challange and I change a bit the padding and the border-radius for a better match.
Wish you all the best!