Skip to content
OverviewSolutions
11
Comments
10
Ataize Feitosa
@Ataize

All comments

  • Ataize Feitosa•200
    @AtaizePosted 24 days ago

    Hello, your project is very good. A small suggestion to improve is to change the email '[email protected]' to the email registered in the <span> tag in your 'thanks-container' div. You can do this using 'inner.Text'

    You're doing great, congratulations!

    0
  • Abhi•320
    @abhi-zeroSubmitted about 1 month ago
    What are you most proud of, and what would you do differently next time?

    This is my first project with JavaScript. Working with JavaScript is difficult for me, but it’s interesting.

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

    A little bit of challenge occurs during development, but I fix most of those issues myself, and I take a little help from GPT.

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

    Mmm, this is my first project, so I don’t know much about how to handle JavaScript. My logic is simple: when user click the button, it adds a class to the share section, which has two wrappers—one for mobile and one for desktop. I created a class called 'hide' with the CSS rule display: hide; (I know, what was I thinking?). This 'hide' class is the initial state, and when the user clicks the button, it removes the 'hide' class and adds the appropriate wrapper class according to the window size.

    I’m sure I’ll improve my skills faster than a squirrel on an espresso! Any feedback or suggestions are welcome!"

    Article Preview |HTML| |CSS| |JS|

    2
    Ataize Feitosa•200
    @AtaizePosted about 1 month ago

    Hello everything is fine?

    I'm just a beginner, but I think you could optimize your javascript by adding/removing classes in a more concise way, using toggle.

    The toggle method toggles the presence of the hidden class in the element every time the button is clicked.

    I hope I helped and keep it up.

    Marked as helpful
    1
  • Ataize Feitosa•200
    @AtaizePosted about 1 month ago

    Congratulations, your code is well organized. I also liked the names you gave the classes, I thought they were very descriptive. Congratulations!

    0
  • Ataize Feitosa•200
    @AtaizePosted about 1 month ago

    Congratulations, your solution was very similar to the one suggested.

    A suggestion to improve your HTML markup when using lists is for each item to use the <li> tag Example:

    <ul>
   <li>first item</li>
   <li>second item</li>
   <li>third item</li>
</ul>

    I hope I helped in some way. Keep studying, you are doing great.

    Marked as helpful
    0
  • Israel Andreotti•270
    @Israel-AndreottiSubmitted about 1 month ago
    What are you most proud of, and what would you do differently next time?

    I'm proud that I was able to make that layout responsive on every device.

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

    I'm it was “thinking outside the box” at first I was picturing the four cards in one container, after I while I divided the card into three columns, then it became very easy to make to layout

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

    I had to search about how to move the images inside the cards, so if you have any tips about that, it would be awesome.

    Four Cards Featura Section

    1
    Ataize Feitosa•200
    @AtaizePosted about 1 month ago

    Congratulations, your work was very good.

    0
  • Ataize Feitosa•200
    @AtaizePosted about 2 months ago

    Congratulations, your project was excellent.

    Marked as helpful
    0
  • Ataize Feitosa•200
    @AtaizePosted about 2 months ago

    Your work was excellent!

    I'm just starting out, but I would suggest you use the <a> tag for links. Using <button> is best suited for actions (e.g. form submissions), while links to external websites should use <a> elements. You can style anchor tags to look like buttons if you want.

    I hope I helped and good luck on your journey!

    Marked as helpful
    0
  • Ataize Feitosa•200
    @AtaizePosted about 2 months ago

    Your project is very good. Congratulations!

    To make it even better, use semantic HTML tags: Instead of using generic <div> tags for main sections of content, you should use more semantic HTML elements like <article>, <header>, and <footer>. This improves accessibility and SEO. These changes will improve the structure and semantics of your HTML, making it more accessible and SEO-friendly while also being easy to maintain.

    I hope I have contributed in some way.

    0
  • Ataize Feitosa•200
    @AtaizePosted 2 months ago

    Congratulations on your project.

    A small suggestion: To improve SEO it is important to use only one <h1> tag per page. Using just one <h1> creates hierarchy and gives the page a more logical structure.

    I hope I helped and once again congratulations on the project.

    0
  • Ataize Feitosa•200
    @AtaizePosted 2 months ago

    Congratulations!

    0