P
YassinMohamed• 510
@yasso2weny
Submitted
I could not add the share button pop-up so any help would be appreciated.
Mostafa Elkhateeb
@melkhateeb193
All comments
- Mostafa Elkhateeb• 710
@melkhateeb193
Posted
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 - PFlávia Reis• 650
@flaviare1s
Submitted
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? :)
Mostafa Elkhateeb• 710@melkhateeb193
Posted
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 - BrovoJD• 190
@Jaheim-Douglas
Submitted
Mostafa Elkhateeb• 710@melkhateeb193
Posted
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 <31 - Nisarga sudeep• 20
@nish8888888888
Submitted
creating responsive images and grid creating responsive images
Mostafa Elkhateeb• 710@melkhateeb193
Posted
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 youMarked as helpful
0 - mostafa abdelzaher• 160
@Mostafa-ninoo
Submitted
give me your opinion
Mostafa Elkhateeb• 710@melkhateeb193
Posted
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
