Frontend Mentor | Parcel + Mithril coding challenge solution

Community feedback

@ApplePieGiraffe

Posted about 3 years ago

Hello, Felipe Oliveira! 👋

Nice to see you complete this challenge! 😀 It is a simple one, but you've done a good job on everything! 👍

I had actually never heard of Mithril.js before, but I think it sounds interesting (haha, especially since it's named, well, "Mithril"). Probably something I should check out sometime! 😅

Of course, keep coding (and happy coding, too)! 😁

Felipe OG• 730

@felipeog

Posted about 3 years ago

Hey, @ApplePieGiraffe!

Hahaha! Mithril is new to me as well.

I wanted to try it out as its docs says it's smaller and performs better than other popular frontend frameworks. I just learned the very basics yet.

It was fun, I recommend you to try it yourself.

Happy coding!

Raymart Pamplona• 16,090

@pikapikamart

Posted about 3 years ago

Hey, good work on this one and for mithril? Hmm never heard about that before.

There are couple of suggestions from me

Excess usage of H1. In your markup, you used multiple h1 tags which is not recommended especially if it is a single page. Changing them to other tags will be great. Following best practices is really good.
Button. Your learn more is supposed to be a link right and I think button element is not the right element to use in here, it is supposed to be a tag.

Aside from those mentioned. Your work is good. It resizes well when going in mobile as well as the layout in the desktop^

Felipe OG• 730

@felipeog

Posted about 3 years ago

Hello, @pikamart! Thanks for taking a look.

As far as I know, it's ok to each article have its own h1, as the specification says. Could you explain why it's not recommended?
I don't think that the layout explicitly shows they're links, they could be buttons to open a modal, for example. But I see that they could be links as well.

Thank you again and happy coding!

Raymart Pamplona• 16,090

@pikapikamart

Posted about 3 years ago

@felipeog - Yes, they need to have each heading tags on its own, like h2 h3, but h1 is supposed to be used only one per page and is usually the top most header text of your webpage. You can use, but 1 h1 per page is considered the best practice.

I was also hesitant for that as well since yes, it could be a modal, but for common it will be a link

Thank you as well ^

Felipe OG• 730

@felipeog

Posted about 3 years ago

Hey, @pikamart! Thanks for the follow up.

The one h1 tag per page rule is valid for HTML versions prior to HTML5. The solution was made with HTML5.

You're right about links being more common, I will change them soon. Thanks!

Keep coding!

Parcel + Mithril

Design comparison

Solution retrospective

Community feedback

Please log in to post a comment

Join our Discord community