Skip to content
OverviewSolutions
4
Comments
4
P
Bruno Banoczi-Cs
@L4r4TW

All comments

  • Agustina•155
    @aguscorvoSubmitted about 2 months ago
    What challenges did you encounter, and how did you overcome them?

    Recipe image and padding in responsive view. I ended up using a div as a wrapper and media queries.

    Recipe Page

    1
    Bruno Banoczi-Cs•40
    @L4r4TWPosted about 2 months ago

    It's a pleasure to see a so perfect solution. The mobile view is also beautiful.

    Making the image responsive was a huge pain for me to, I copy my mediaquery selection, hopefully it will show you something helpful:

    /* Remove border radius on mobile screens */
@media (max-width: 768px) {
  /* Adjust the breakpoint as needed */
  .container {
    border-radius: 0; /* Remove border radius on mobile */
    margin: 0;
    padding-top: 0;
    /* box-sizing: content-box; */
  }

  .container img {
    border-radius: 0;
    width: 100vw;
    height: auto;
    margin-left: calc((-100vw + 100%) / 2);
  }
}
    0
  • Bruno Banoczi-Cs•40
    @L4r4TWPosted 2 months ago

    Hi Christian! I love your solution😍

    Here are some tips:

    • Try to use semantic HTML tags instead of <div>-s because ti makes your work more professional

    • The buttons in this project aren't real buttons, they are links that looks like buttons. Using links makes it much easyer to redirect to the given page, and if you hoover the cursor, it will automatically change to pointer, which is a small thing, but increases strongly the user experience.

    Here is an example:

     <a
    href="https://linkedin.com"
    class="button-link"
    target="_blank"
    aria-label="Twitter"
>Twitter</a>

    .button-link {
  display: flex;
  justify-content: center;
  align-items: center; /* Centers vertically */
  width: 304px;
  height: 45px;
  margin-bottom: var(--spacing-lg);
  font-size: 16px; /* Set font size */
  color: #fff; /* Text color */
  background-color: var(--grey-700); /* Background color (blue) */
  text-align: center; /* Center text inside button */
  text-decoration: none; /* Remove underline */
  border-radius: 8px; /* Rounded corners */
}
    • Hoovering over your buttons has a good looking color changing effect. You can make it better by changing the lenght of the effect:
    .button-link{
    transition: background-color 2s ease; /* Smooth color transition */
}
    • If you want to give a function that makes it able to use the arrow keys to navigate between the social media links you can use this small JavaScript code:
    // script.js

document.addEventListener("keydown", function (event) {
  // List of buttons
  const buttons = document.querySelectorAll(".button-link");

  // Get the index of the currently focused button
  const focusedButtonIndex = Array.from(buttons).indexOf(
    document.activeElement
  );

  if (event.key === "ArrowDown") {
    // Move to the next button (down arrow key)
    const nextIndex = (focusedButtonIndex + 1) % buttons.length;
    buttons[nextIndex].focus();
    event.preventDefault(); // Prevent default behavior (scrolling, etc.)
  } else if (event.key === "ArrowUp") {
    // Move to the previous button (up arrow key)
    const prevIndex =
      (focusedButtonIndex - 1 + buttons.length) % buttons.length;
    buttons[prevIndex].focus();
    event.preventDefault(); // Prevent default behavior
  }
});

    Hope I gave you some helpful information, not just bullshit☺️ and wish you a lot of fun with the future projects!!

    0
  • Bruno Banoczi-Cs•40
    @L4r4TWPosted 3 months ago

    Hi Yessica!

    You made a pretty good looking card.

    The only tip I can give you as a beginner is to avoid using style attributes in your HTML code, because it make the code messy. Put all style attributes into the CSS file.

    Also consider using more whitespace between the elements, because easy to read content is vital in webdesign.

    Keep up the good job! :)

    0
  • Bruno Banoczi-Cs•40
    @L4r4TWPosted 3 months ago

    This is a perfect solution!

    0