Skip to content
OverviewSolutions
39
Comments
5
Mostafa Elkhateeb
@melkhateeb193

All comments

  • Mostafa Elkhateeb•710
    @melkhateeb193Posted 10 months ago

    you Can make a button share in <div> and give if position relative and in it make two divs one is the share button without details and the other one make new div and give it position absolute and put on in all the data you want to add and when click

    0
  • Flávia Reis•650
    @flaviare1sSubmitted 10 months ago

    I had a lot of difficulty doing it, as I'm still at the beginning of my JavaScript studies. I couldn't create a code that covered all the questions, I had to create separate ids. Can anyone help me and give me feedback? :)

    Responsive landing page using media query

    2
    Mostafa Elkhateeb•710
    @melkhateeb193Posted 10 months ago

    that a great job you have done you can make it shorter code you can give to the plus item same id or same class and use query selector all const faqItem = document.querySelectorAll('.faq-item'); like this faqItem.forEach(function(item) { const toggleContent = item.querySelector('.toggle-content'); const plus = item.querySelector('.plus'); const minus = item.querySelector('.minus');

    plus.addEventListener('click',function(){ plus.classList.replace('d-block','d-none'); toggleContent.classList.replace('d-none', 'd-block') minus.classList.replace ('d-none', 'd-block') })

    minus.addEventListener('click',function(){ plus.classList.replace('d-none', 'd-block'); toggleContent.classList.replace('d-block','d-none') minus.classList.replace ('d-block','d-none') }) }) that will work with you and with less could and keep going you have done a great job

    Marked as helpful
    0
  • Mostafa Elkhateeb•710
    @melkhateeb193Posted 10 months ago

    first great job you did and second you need to make a main container to save all the element you made on it and give to the container width and height depends on the card size

    <main> but here all the content<main/> and here modify what ever you want to do i hope that will fix your problem and you can give the body 100vh to make it take all the desktop height and again great job keep coding <3
    1
  • Mostafa Elkhateeb•710
    @melkhateeb193Posted almost 2 years ago

    that a great job you have done here but there some mistake i hope that you accept it from me

    <h6>P E R F U M E</h6> instead of use <h1> PERFUME</h1> and you have used more than h6 in the same design and it not recommended in the web and google fonts is recommended to be used in the css sheet not in HTML and its optional <button type="button" class="btn"><img src="images\icon-cart.svg"> Add to Cart</button> the button should lead somewhere so put the same button inside <a></a> and increase the padding of the text hope thats helps you
    Marked as helpful
    0
  • Mostafa Elkhateeb•710
    @melkhateeb193Posted almost 2 years ago

    its really got design a wish all the best but 1st of all the button is going some where so you have to put it in
    <a href="#"> <button> Proceed to Payment</button></a> tag and the 2nd thing is optional its better to put the google fonts tags in css- and same to the images is better for web

    Marked as helpful
    0