Skip to content
OverviewSolutions
13
Comments
14
Branka R
@brainka

All comments

  • Achara Chisom Solomon•490
    @AcharaChisomSolomonSubmitted 7 months ago
    What are you most proud of, and what would you do differently next time?

    Whewww!!! . . . . This looked deceptively simple at first, definitely worthy of being an intermediate ranked project!!!

    getting the strength display was fun and is what I am most proud of.

    What challenges did you encounter, and how did you overcome them?

    A lot like figuring out how copying to the clipboard works, determining the strength of a password using regex and the css tricks involves, and also overriding the default styling for html input elements.

    loads of googling and chatgpt helped.

    What specific areas of your project would you like help with?

    nothing for now!!!

    Any tips to improve is welcome

    Password generator app

    2
    Branka R•290
    @brainkaPosted 5 months ago

    Hey!

    I totally agree, it was very deceptive but equally great and challenging.

    Really good job on this project :)

    1
  • Graeme Byrne•520
    @grmbyrnSubmitted 6 months ago
    What are you most proud of, and what would you do differently next time?

    Getting the JS to all work together.

    What challenges did you encounter, and how did you overcome them?

    I found the layout of the project a little strange. Having an error message appear when there isn't a button to press to make the calculation was a little odd for me. I left it out.

    What specific areas of your project would you like help with?

    I would like any advice on how to get the bill, tip percentage, and number of people info and how best to calculate the tip amount and total as I think I could improve that.

    Tip calculator

    1
    Branka R•290
    @brainkaPosted 6 months ago

    Hey! Nice work on project.

    From my observations, your calculations appear to correctly calcualte the tips, so I don't believe you need to change anything there.

    Just a few items of note:

    As a user going through your form, I notice that the tip amount and tip total only evaluate when you click on the select tip % button or when you enter a custom amount.

    Which means, when the user enters in the Bill, selects the tip % and enters the number of people... nothing happens....the user has to then go back to click on the select tip % in order to have the total evaluate.

    I would consider adding an additional event listeners to your other input fields (just like you did with the custom tip field), so when the user types or updates their values, the form will also be updating the totals.

    Hope that helps a bit!

    Happy coding :)

    Marked as helpful
    0
  • Branka R•290
    @brainkaPosted 6 months ago

    Hey!

    Nice job on the project :)

    0
  • João Vitor Morita•210
    @joaomoritaSubmitted 7 months ago
    What are you most proud of, and what would you do differently next time?

    I'm proud to use jquery in this project, it was my first "junior" project. I think I can do better my html content, using more semantics tags

    What challenges did you encounter, and how did you overcome them?

    The main challenge was the email validation, I went trought the internet looking for some solutions to overcome this one. The second challenge was give the feedback with colors when the email was invalid, this time I need to search a little bit about some jquerys funcions like "addClass" and "removeClass"

    What specific areas of your project would you like help with?

    I'm looling forward to see some feedbacks about my code in general to improve my skills. Thank you guys!

    Responsive newsletter with jquery

    1
    Branka R•290
    @brainkaPosted 6 months ago

    Hey!

    Well done on completing this project :)

    Just a few small items I noticed on your project:

    • You are missing the main element in your html. Your container class should be a main element instead of a div.

    • You have missed adding a background colour to your form button:hover{ cursor: pointer; } when you hover over it.

    Again, good job :)

    Marked as helpful
    1
  • Willen Garcia•170
    @willengarciaSubmitted 6 months ago
    What are you most proud of, and what would you do differently next time?

    Me orgulho de ter conseguido fazer esse desafio, pra mim, apesar de ser simples, tive algumas dificuldades, porém consegui superar algumas delas, tais como o uso do JS, e o desing responsivo para do drop-up ficar bonito.

    What challenges did you encounter, and how did you overcome them?

    Responsividade do leiaute

    What specific areas of your project would you like help with?

    nenhuma

    Grid, Flexbox, JS

    1
    Branka R•290
    @brainkaPosted 6 months ago

    Goos job on your project!

    The responsiveness is working pretty well :)

    0
  • Michael•380
    @MlchaellSubmitted over 1 year ago

    I wasn't sure how to do the hero image properly so they are there, but the sizing/placement isn't right. If anyone would have any ideas let me know!

    Thanks

    Meet Landing Page (Vanilla CSS)

    1
    Branka R•290
    @brainkaPosted 6 months ago

    Hey!

    Really good job with the project.

    I would suggest looking at css position property. This is how I managed to move the placement of the images.

    Question: Why did you not use the <img> element instead of showing the image as a psuedo element?

    What you could do is wrap the hero image/s in an <img> element then add for example: position: relative top: -10px

    This will move the image from its position down by 10px.

    You play around with this with your section class="img-grid" container to see how it works!

    Hope you find this useful 😀

    Please let me know how you go!

    0
  • Branka R•290
    @brainkaSubmitted 7 months ago
    What are you most proud of, and what would you do differently next time?

    Proud that I managed to get the desired grid layout.

    Next time, I would plan it out ahead of time before starting the code.

    What challenges did you encounter, and how did you overcome them?

    I resort to using flex by default, so made myself use grid for the entire layout.

    No issues, just need more practice using grid layout.

    What specific areas of your project would you like help with?

    Just overall code review and one question.

    I understand that the accessibility report is highlighting that the page requires a single h1 tag.

    In this instance, this isn't really a landing page, but another section and this is the reason I have not added any h1 elements.

    Is this correct?

    Thank you

    Responsive Testimonals Grid Section

    2
    Branka R•290
    @brainkaPosted 6 months ago

    Hey,

    Thank you :)

    Will consider that for sure!

    0
  • Abdelrahman mohamed•230
    @Tayara97Submitted 7 months ago
    What are you most proud of, and what would you do differently next time?

    .

    What challenges did you encounter, and how did you overcome them?

    it was challenging for me when i start with mobile-first but i did it well

    What specific areas of your project would you like help with?

    ..

    responsive Testimonials grid

    1
    Branka R•290
    @brainkaPosted 7 months ago

    Good work!

    1
  • Fish Ladder•140
    @fish-ladderSubmitted 7 months ago
    What are you most proud of, and what would you do differently next time?

    I am proud of going a step further with the responsive design and adding a mid-size screen breakpoint with a third layout of the 4 cards.

    I was also proud to accurately dial in the little colored accent lines at the top of each card and get the box-shadow effect very close to the design.

    Next time I will try to pace myself better. I got a bit manic with my styles and started getting sloppy with writing my code. This made more work for me later in tidying things up.

    What challenges did you encounter, and how did you overcome them?

    I had a much harder time getting the colored accent stripe right on the top of each card. I first tried just a top border but it didn't look right. Then I tried creating a separate div inside the card but that didn't achieve the right look either. Finally, I created a linear gradient in the background and I think that did the trick.

    I also spent more time that I would have like tweaking details like margins and padding in the desktop layout.

    What specific areas of your project would you like help with?

    In terms of layout and sizing, where is it OK to use absolute units like px and when should I be using em or rem?

    Four card feature section use grid and media queries

    1
    Branka R•290
    @brainkaPosted 7 months ago

    Hey,

    Really good job on your project and the way you have structured your html and css 😄

    I don't like to just paste links but I find this explanation very useful when I forget when to use px, rem or em haha ===> EM vs REM vs PX.

    Hope this helps!

    Good job again 🥳

    Marked as helpful
    1
  • ajkun55•1,400
    @ajkun55Submitted 7 months ago
    What are you most proud of, and what would you do differently next time?

    all

    What challenges did you encounter, and how did you overcome them?

    ok

    What specific areas of your project would you like help with?

    ok

    Product-preview-card-component

    1
    Branka R•290
    @brainkaPosted 7 months ago

    Hello!

    Nice job! The comparison is really close :)

    I have noticed you used margin to add spacing to your text div.

    What you can do instead is add padding next time. Padding is used to create space around the content, where as margin is used to create space between elements.

    Just something to keep in mind for next time!

    Again, really good job :)

    Marked as helpful
    0
  • ralrafie96•80
    @ralrafie96Submitted 7 months ago
    What are you most proud of, and what would you do differently next time?

    I am glad I could structure the document into elements effectively

    What challenges did you encounter, and how did you overcome them?

    I was unsure how to properly style and structure the list items and table components

    What specific areas of your project would you like help with?

    None for now, I feel comfortable with the project so far

    Recipe Page Main

    1
    Branka R•290
    @brainkaPosted 7 months ago

    Hey!

    Just a tiny note. Consider splitting your html and css files separately. It makes it easier to read and won't clutter the html file :)

    Otherwise, good work on the component :)

    0
  • Srithar_K•150
    @SritharanKalimuthuSubmitted 7 months ago
    What are you most proud of, and what would you do differently next time?

    Nothing

    What challenges did you encounter, and how did you overcome them?

    None

    What specific areas of your project would you like help with?

    Nothing

    Social Profiles Links

    1
    Branka R•290
    @brainkaPosted 7 months ago

    Keep up the good work! Looks pretty spot on!

    Marked as helpful
    0