Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create Mushi's Henna Website #701

Closed
wants to merge 1 commit into from
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
231 changes: 231 additions & 0 deletions Mushi's Henna Website
Original file line number Diff line number Diff line change
@@ -0,0 +1,231 @@
<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>Mushi's Henna</title>
<style>
body {
font-family: 'Verdana', sans-serif;
margin: 0;
padding: 0;
background-color: #FFF8F0;
}

header {
background-color: #944D60;
color: white;
padding: 20px;
text-align: center;
}

nav {
display: flex;
justify-content: center;
gap: 20px;
background-color: #E39A9A;
padding: 15px;
}

nav a {
text-decoration: none;
color: white;
font-weight: bold;
font-size: 1.1rem;
}

nav a:hover {
text-decoration: underline;
}

.hero {
text-align: center;
padding: 60px 20px;
background-image: url('your-hero-image.jpg');
background-size: cover;
background-position: center;
color: black;
}

.hero h1 {
font-size: 3.5rem;
margin-bottom: 10px;
color: black;
}

.hero p {
font-size: 1.2rem;
color: black;
}
Comment on lines +41 to +59
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue

Fix hero section implementation

  1. The background image URL is invalid (your-hero-image.jpg)
  2. Text readability might be compromised depending on the background image

Apply these changes:

 .hero {
     text-align: center;
     padding: 60px 20px;
-    background-image: url('your-hero-image.jpg');
+    background-image: url('./images/hero-bg.jpg');
     background-size: cover;
     background-position: center;
-    color: black;
+    position: relative;
+    z-index: 1;
 }
+
+.hero::before {
+    content: '';
+    position: absolute;
+    top: 0;
+    left: 0;
+    right: 0;
+    bottom: 0;
+    background: rgba(255, 255, 255, 0.7);
+    z-index: -1;
+}
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
.hero {
text-align: center;
padding: 60px 20px;
background-image: url('your-hero-image.jpg');
background-size: cover;
background-position: center;
color: black;
}
.hero h1 {
font-size: 3.5rem;
margin-bottom: 10px;
color: black;
}
.hero p {
font-size: 1.2rem;
color: black;
}
.hero {
text-align: center;
padding: 60px 20px;
background-image: url('./images/hero-bg.jpg');
background-size: cover;
background-position: center;
position: relative;
z-index: 1;
}
.hero::before {
content: '';
position: absolute;
top: 0;
left: 0;
right: 0;
bottom: 0;
background: rgba(255, 255, 255, 0.7);
z-index: -1;
}
.hero h1 {
font-size: 3.5rem;
margin-bottom: 10px;
color: black;
}
.hero p {
font-size: 1.2rem;
color: black;
}


.products {
display: grid;
grid-template-columns: repeat(auto-fit, minmax(250px, 1fr));
gap: 20px;
padding: 20px;
background-color: #FFEFE8;
}

.product {
background-color: white;
padding: 20px;
box-shadow: 0 2px 8px rgba(0, 0, 0, 0.1);
border-radius: 8px;
text-align: center;
}

.product img {
max-width: 100%;
border-radius: 8px;
}

.product h3 {
margin: 15px 0 10px;
color: #944D60;
}

.product p {
color: #555;
font-size: 1rem;
}

.about, .contact {
padding: 40px 20px;
background-color: #FFF8F0;
text-align: center;
}

.about h2, .contact h2 {
color: #944D60;
}

.contact-icons {
display: flex;
justify-content: center;
gap: 15px;
margin: 20px 0;
}

.contact-icons a {
color: #944D60;
font-size: 1.5rem;
text-decoration: none;
}

.contact-icons a:hover {
color: #E39A9A;
}

footer {
background-color: #944D60;
color: white;
text-align: center;
padding: 20px;
}

footer a {
color: #FFD700;
text-decoration: none;
}

footer a:hover {
text-decoration: underline;
}

i {
font-family: "FontAwesome", sans-serif;
}
Comment on lines +135 to +137
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue

Fix Font Awesome icon implementation

The current Font Awesome implementation won't work correctly. The i tag styling alone isn't sufficient, and the script loading approach is outdated.

Replace with the current Font Awesome CDN:

-    i {
-        font-family: "FontAwesome", sans-serif;
-    }
-</style>
-<script src="https://kit.fontawesome.com/a076d05399.js" crossorigin="anonymous"></script>
+</style>
+<link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/font-awesome/6.5.1/css/all.min.css">

Committable suggestion skipped: line range outside the PR's diff.

</style>
Comment on lines +7 to +138
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion

Move styles to a separate CSS file for better maintainability

Consider extracting styles to a separate CSS file to improve:

  • Code organization
  • Caching
  • Maintainability

Create a new file styles.css and replace the style tag with:

-<style>
-    /* current styles */
-</style>
+<link rel="stylesheet" href="styles.css">

Committable suggestion skipped: line range outside the PR's diff.

<script src="https://kit.fontawesome.com/a076d05399.js" crossorigin="anonymous"></script>
</head>
<body>
<header>
<h1>Mushi's Henna</h1>
<p>Creating Beautiful Moments with Henna</p>
</header>

<nav>
<a href="#products">Products</a>
<a href="#about">About Us</a>
<a href="#contact">Contact</a>
</nav>

<section class="hero">
<h1>Welcome to Mushi's Henna</h1>
<p>Explore our premium range of henna products and designs crafted with love</p>
</section>

<section id="products" class="products">
<div class="product">
<img src="C:\Users\AZAR\Desktop\henna-website\images\Natural Henna Cones - 1.jpg" alt="Natural Henna Cone"width="500" height="300">
<h3>Natural Henna Cone</h3>
<p>Rs.300</p>
</div>
<div class="product">
<img src="C:\Users\AZAR\Desktop\henna-website\images\Aftercare Balm - 2.jpg" alt="Aftercare Balm"width="500" height="300">
<h3>Aftercare Balm</h3>
<p>Rs.170</p>
</div>
<div class="product">
<img src="C:\Users\AZAR\Desktop\henna-website\images\Aftercare Sealant Spray.jpg" alt="Aftercare Sealant Spray"width="500" height="300">
<h3>Aftercare Sealant Spray</h3>
<p>Rs.250</p>
</div>
<div class="product">
<img src="C:\Users\AZAR\Desktop\henna-website\images\Henna Powder - 1.jpg" alt="Henna Powder"width="500" height="300">
<h3>Henna Powder</h3>
<p>100g - Rs.600<br>1Kg - Rs.4,900</p>
</div>
<div class="product">
<img src="C:\Users\AZAR\Desktop\henna-website\images\Eucalyptus Oil.jpg" alt="Eucalyptus Oil"width="500" height="270">
<h3>Essential Oils: Eucalyptus Oil</h3>
<p>100ml - Rs.1,650</p>
</div>
<div class="product">
<img src="C:\Users\AZAR\Desktop\henna-website\images\Tea Tree Oil.jpg" alt="Tea Tree Oil"width="500" height="270">
<h3>Essential Oils: Tea Tree Oil</h3>
<p>100ml - Rs.1,850</p>
</div>
<div class="product">
<img src="C:\Users\AZAR\Desktop\henna-website\images\Blend Oil.jpg"alt="Signature Blend Oil"width="500" height="300">
<h3>Essential Oils: Signature Blend Oil</h3>
<p>100ml - Rs.1,700</p>
</div>
<div class="product">
<img src="C:\Users\AZAR\Desktop\henna-website\images\Practice Henna Cones - 1.jpg" alt="Practice Henna Cones"width="500" height="300">
<h3>Practice Henna Cones</h3>
<p>Each - Rs.180<br>Buy 6 for Rs.980</p>
</div>
</section>
Comment on lines +158 to +199
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue

Critical: Fix product implementation issues

  1. The absolute file paths (C:\Users\AZAR\Desktop\...) will break in production

  2. Missing cart functionality mentioned in PR objectives

  3. No way for users to purchase products

  4. Use relative paths for images:

-<img src="C:\Users\AZAR\Desktop\henna-website\images\Natural Henna Cones - 1.jpg"
+<img src="./images/natural-henna-cones-1.jpg"
  1. Add cart functionality to each product:
 <div class="product">
     <img src="./images/natural-henna-cones-1.jpg" alt="Natural Henna Cone">
     <h3>Natural Henna Cone</h3>
     <p>Rs.300</p>
+    <div class="product-actions">
+        <input type="number" min="1" value="1" class="quantity-input">
+        <button class="add-to-cart" data-product-id="1" data-price="300">
+            Add to Cart
+        </button>
+    </div>
 </div>
  1. Add cart UI and functionality:
<div id="cart" class="cart">
    <h2>Shopping Cart</h2>
    <div id="cart-items"></div>
    <div class="cart-total">
        <strong>Total: </strong>
        <span id="cart-total">Rs.0</span>
    </div>
    <button id="checkout">Proceed to Checkout</button>
</div>

Would you like me to provide the complete cart implementation including the JavaScript code?


<section id="about" class="about">
<h2>About Us</h2>
<p>Welcome to Mushi's Henna, where passion meets artistry. My name is Mushrifa Moulana, and I have been a dedicated henna artist since the age of 16. What began as a simple hobby quickly blossomed into a lifelong passion for creating intricate designs and crafting premium henna products.
</p>
<p>
At Mushi's Henna, we believe that henna is more than just a temporary adornment—it is a celebration of culture, beauty, and individuality. Each product we offer is meticulously formulated using natural ingredients to ensure safety, quality, and vibrant results. Whether you are preparing for a special occasion or simply indulging in self-expression, our range of henna cones, powders, and aftercare products are designed to make your experience truly special.
</p>
<p>
What sets us apart is not just our commitment to quality but also the love and care that goes into every design and product. My journey as a henna artist has been deeply fulfilling, and I am thrilled to share this passion with all of you. Thank you for choosing Mushi's Henna as your trusted partner in celebrating the art of henna.
</p>
</section>

<section id="contact" class="contact">
<h2>Contact Us</h2>
<p>Mushrifa Moulana</p>
<p>Call or Whatsapp - <a href="tel:+94783921216">+94 78 39 21216</a></p>
<p>Instagram - <a href="https://www.instagram.com/mushi.s_henna" target="_blank">mushi.s_henna</a></p>
<div class="contact-icons">
<a href="https://wa.me/94783921216" target="_blank"><i class="fab fa-WhatsApp"></i></a>
<a href="https://www.instagram.com/mushi.s_henna" target="_blank"><i class="fab fa-Instagram"></i></a>
<a href="mailto:[email protected]" target="_blank"><i class="fas fa-envelope"></i></a>
</div>
<p>Have questions? Reach out to us at <strong>[email protected]</strong> or follow us on social media!</p>
</section>
Comment on lines +213 to +224
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue

Address security concerns in contact information

  1. Direct exposure of personal contact details poses privacy risks
  2. WhatsApp icon class is incorrectly capitalized

Consider these changes:

-<p>Call or Whatsapp - <a href="tel:+94783921216">+94 78 39 21216</a></p>
+<p>Call or WhatsApp - <button onclick="showContact()" class="contact-btn">Click to Show</button></p>

-<a href="https://wa.me/94783921216" target="_blank"><i class="fab fa-WhatsApp"></i></a>
+<a href="https://wa.me/94783921216" target="_blank"><i class="fab fa-whatsapp"></i></a>

-<p>Have questions? Reach out to us at <strong>[email protected]</strong></p>
+<p>Have questions? <a href="#" onclick="showEmail()" class="contact-btn">Click to Show Email</a></p>

Add JavaScript to handle contact information:

function showContact() {
    document.querySelector('.contact-btn').textContent = '+94 78 39 21216';
}

function showEmail() {
    event.preventDefault();
    document.querySelector('.contact-btn').textContent = '[email protected]';
}


<footer>
<p>&copy; 2024 Mushi's Henna. All rights reserved.</p>
<p><a href="#">Terms of Service</a> | <a href="#">Privacy Policy</a></p>
</footer>
Comment on lines +226 to +229
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion

Implement missing footer functionality

  1. Terms and Privacy Policy links are empty
  2. Copyright year is hardcoded

Apply these changes:

-    <p>&copy; 2024 Mushi's Henna. All rights reserved.</p>
-    <p><a href="#">Terms of Service</a> | <a href="#">Privacy Policy</a></p>
+    <p>&copy; <script>document.write(new Date().getFullYear())</script> Mushi's Henna. All rights reserved.</p>
+    <p><a href="./terms.html">Terms of Service</a> | <a href="./privacy.html">Privacy Policy</a></p>

Would you like me to provide templates for the Terms of Service and Privacy Policy pages?

Committable suggestion skipped: line range outside the PR's diff.

</body>
</html>
Loading