Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Log in with GitHub
OverviewSolutions11Comments13

All comments

  • Solomon Ekrebe• 610

    @krebeDev

    Posted

    I appreciate your feedback @mattstuddert.

    As a beginner, working with React was quite tough, but exciting nonetheless.

    • I didn't know assigning e.currentTarget.id to userChoice.name will directly mutate the userChoice object in state, because I started off by cloning userChoice and houseChoice like so:
    handlePick = (e) => {
    const userChoice = {...this.state.userChoice};
    const houseChoice = {...this.state.houseChoice};

    // Other code lines go here
}

    Do I have to clone/setState for each key : value pair in this instance?

    I've started learning about hooks, and definitely will apply them once I get the hang of it.

    • On the identical components, I'll refactor my code as you suggested. I just noticed the same repetitive pattern in my Starter component too. I'll fix them.
    0
  • Solomon Ekrebe• 610

    @krebeDev

    Submitted

    I've had to use two empty divs to display the curly background patterns in the intro section. I feel this approach is not effective. Any ideas on a better way to achieve the design that's in line with best design practices? You're welcome.

    Insure landing page with - Vanilla JS, HTML and CSS

    2

    Solomon Ekrebe• 610

    @krebeDev

    Posted

    Thanks for the feedback @mattstuddert. I didn't know I could use url() as content for ::before and ::after pseudo-elements until today. I'll definitely give it a try.

    0
  • Solomon Ekrebe• 610

    @krebeDev

    Posted

    The design is great on desktop/large screens. I suggest you include media queries by introducing different breakpoints/CSS rules to make it responsive in smaller viewports.

    1
  • Solomon Ekrebe• 610

    @krebeDev

    Posted

    I see nothing missing :). Great job you've done here!

    1
  • faheem102• 38

    @faheem102

    Submitted

    hej this is my solution as a beginner. i do not know how to fill logo color into white. can some one please explain to me with some easy method to it. thank you

    fylo landing page

    1

    Solomon Ekrebe• 610

    @krebeDev

    Posted

    Hey @faheem102,

    Great work on this challenge. To change the logo colour, one way is to load it directly as svg element instead of using img tag. Then change the fill color of the svg to white inside the svg style definition.

    1
  • vongphakdy• 140

    @vongphakdy

    Submitted

    I have 2 error cases for email validation; the first is an empty field, second is an invalid email address. Both cases should activate the error message and icon, however only this only applies to the first case. When entering an invalid input, the form reveals a prompt rather than my coded error message/icon. I'm new to JS so I appreciate any input

    Email validation with VanillaJS, SCSS, HTML

    2

    Solomon Ekrebe• 610

    @krebeDev

    Posted

    Hello @vongphakdy, Great job on this challenge. Your error message is not being activated because HTML5 form comes with built-in form validation. To deactivate this default behaviour and ensure your JavaScript code is triggered, add a novalidate attribute on the form element. In your case, your code will be like so <form novalidate class="main-form">.

    Also, your JS form validation code should be enhanced to accommodate other use cases. For example, if a user enters a correct email after a wrong attempt, the error message should be discarded.

    2
  • Solomon Ekrebe• 610

    @krebeDev

    Posted

    Wow!!! Great work Daniel. I personally have been battling with this challenge for some days now. Your code is clean and comprehensible. I have no improvements to recommend. Great job.

    1
  • Solomon Ekrebe• 610

    @krebeDev

    Posted

    I checked in. My observations are the same issues being addressed by @Mattstuddert, particularly the breakpoints. You can improve in those areas.

    1
  • Solomon Ekrebe• 610

    @krebeDev

    Posted

    I also noticed the link to your live site is broken.

    0
  • Solomon Ekrebe• 610

    @krebeDev

    Posted

    Hi Chadrack, great work! Do check your page's Report and fix the accessibility and validation issues. I can help if you need more clarifications on these.

    1
  • Matěj Kubíček• 130

    @matejkubicek

    Submitted

    I'm newbie in front-end development and I know, that my workflow is little bit complicated for this small project, but I trying to learn frontend technologies and tools more comprehensively.

    Please leave me a feedback about my code and used workflow. Thx

    Four card - mobile first, responsive

    2

    Solomon Ekrebe• 610

    @krebeDev

    Posted

    Looks good. But do check the HTML validation report on your solution. The roles you specified in your markup are unnecessary since you already used the appropriate HTML5 semantic elements.

    1
  • Solomon Ekrebe• 610

    @krebeDev

    Posted

    Your work is clean and beautiful. However, in addition to the HTML5 semantic elements already suggested by DJH-dev, you should have used tags like headings (h1,..h6) and <p> for accessibility.

    1