@PremKhodke-99
Submitted
@pertrai1
@PremKhodke-99
Submitted
@pertrai1
Posted
This is great work you have done. It basically aligns with the layout given and has clean markup for both the HTML and CSS. Well done!
@ajibade-dev
Submitted
The h1 element was a little bit tricky for me. I wanted to use the <br /> tag to make the words jump to the next line,but it also did so for medium screens...
@pertrai1
Posted
I like your solution more than the actual design. Very well done.
@pertrai1
Submitted
@pertrai1
Posted
Still working on my code reusability, mainly with CSS. Wondering if putting a lot of values in variables would be helpful or not. Same with properties that are shared across rules, would it be better to combined the two or more rules, or create a helper rule?
@JCastelli12
Submitted
Need help getting @font-face to work with Github Pages! Works locally but won't work when hosted!
@pertrai1
Posted
The code I looked at looks great. I did look and see that there is a 404 for the font location. Maybe changing src: url(/assets/fonts/Inter-VariableFont_slnt\,wght.ttf);
to src: url(assets/fonts/Inter-VariableFont_slnt\,wght.ttf);
where /assets
becomes assets
without the leading /
?
Marked as helpful
@Bezies
Submitted
No particular difficulties for this small project. Except for a small detail, I'm having trouble with the size of my input and my button, I can't find a correct and simple setting so that they have the same size, any idea?
@pertrai1
Posted
Good work with the solution. When you say that you are having trouble with the size of the input and button, do you mean that it would be nice to only write code once and not have to duplicate for both? If so, you could take what is used for both of them and combine into one rule, i.e.
#email,
#sub {
padding: 15px 20px;
}
Then you can remove that from each individual rule. This way, you can change the value once and both will pick it up.
I hope that is what you meant when you need to find a way to have them the same size.
Marked as helpful
@codeguy9
Submitted
Hello, The javascript that I wrote doesn't work. Please if you know more javascript, let me know or put a solution here. Thanks
@pertrai1
Posted
Great work on the solution you have come up with.
Check in your JS and notice that you are using querySelector
for the selected-tab
, but there are multiple elements in the DOM that have this class. Because of this you will need to use querySelectorAll
and then loop over each of them to add the class inactive
. See if that helps get you going in the right direction.
Marked as helpful
@TiffaJenkins
Submitted
This was a nice little challenge, Not too difficult and loved making the slide Nav for mobile, any feedback is welcomed, Thanks guys.
@pertrai1
Posted
Great work on the solution you came up with. A quick look at the code and what I see that could help you:
Do not use the same id
multiple times. I see id="mob-nav-item"
used for all links under .mobile-nav
.
Not much feedback but I hope it is enough to help you continue improving.
Marked as helpful
@GreenT2012
Submitted
Hey,
I’m unsure about trying to use the icons etc. as the data.json file. I guess my solution isn’t the best way so I left it at a try. Maybe someone can show me a better solution.
My structure in the CSS file could also leave something to be desired. It would be great if I could get some feedback on this.
Thank you!
@pertrai1
Posted
I personally think your solution is a good one. Don't think that your solution is not the best because that is very subjective and the objective is to practice. Good work.