Skip to content

Conversation

@KausarShangareeva
Copy link

@KausarShangareeva KausarShangareeva commented Sep 25, 2025

Hi there! πŸ‘‹
I’m currently building DMian, a real estate website focused on the Emirates. The project showcases property listings in a clean, modern design, fully responsive across devices.

Check out the live website here: DMian

Key Features:

  • Clean, green-themed design for a fresh look
  • Fully responsive layout using Flexbox
  • Interactive elements with JavaScript for a better user experience
  • Structured with HTML & CSS for simplicity and maintainability

This project helped me practice responsive layouts, interactivity, and combining HTML, CSS, and JavaScript to create a functional and visually appealing website. 🌟

Copy link
Contributor

@HIPPIEKICK HIPPIEKICK left a comment

Choose a reason for hiding this comment

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

Good job with this business site, Kausyar, looks really nice!
However, it seems like you're missing some requirements:

  • Responsive hamburger menu that's visible only on mobile devices
  • You have divs/spans that look like a form, but I can't see any actual form element which is one of the requirements this week

To add to this, I'm having a hard time checking some of the requirements due to the large amount of files in your repository. Would you mind removing the node_modules (are you using them?) and/or creating a .gitignore file where you write node_modules to make sure they are excluded on GitHub.

Also, there's a little bug when I click the AUD button, then some text appears on top of the existing text. So please also fix this.
Image

Copy link

@mikaelasturk mikaelasturk left a comment

Choose a reason for hiding this comment

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

I really like the clear and detailed comments, especially in the HTML file! It makes the code easy to follow and understand.

It would be great to see a few more comments in the CSS as well, and preferably in English to keep everything consistent. ;)

I also noticed that the hamburger menu is missing from the project, which is part of the requirements. :)

Overall, great work!! The website feels advanced and thoughtfully built!

/* ------------------------------------------------------
| THEME VARIABLES: Π±Π°Π·ΠΎΠ²Ρ‹Π΅ (light) β˜€οΈ
------------------------------------------------------ */
:root {

Choose a reason for hiding this comment

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

Good practice to use root!

Comment on lines +260 to +267
body {
opacity: 0;
transition: opacity 0.3s ease-in-out;
}

body.loaded {
opacity: 1;
}

Choose a reason for hiding this comment

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

This was a really cool detail to add!

css/style.css Outdated
Comment on lines 42 to 57
/* -------------------------------
| 1️⃣ GRID FOR BASE DESKTOP 1️⃣ |
------------------------------- */
/* ---------------------------------------------
| 2️⃣ GRID FOR Tablet landscape: ≀ 1200px 2️⃣ |
-------------------------------------------- */
/* ---------------------------------------------
| 3️⃣ GRID FOR Tablet portrait: ≀ 900px 3️⃣ |
-------------------------------------------- */
/* ---------------------------------------
| 4️⃣ Phone: ≀ 600px Π“ΠžΠ Π˜Π—ΠžΠΠ’ΠΠ›Π¬ΠΠΠ― 4️⃣ |
-------------------------------------- */
/* -------------------------------------
| 5️⃣ Extra small phone: ≀ 320px 5️⃣ |
------------------------------------ */
/*

Choose a reason for hiding this comment

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

Did you forget to put these in the right place? I love the clear, big comments!

Comment on lines +195 to +207
const toggle = document.getElementById("language-toggle");
toggle.addEventListener("click", () => {
localStorage.setItem("scrollPos", window.scrollY);

const currentLang = window.location.pathname.includes("index.en.html")
? "en"
: "ru";
if (currentLang === "ru") {
window.location.href = "index.en.html";
} else {
window.location.href = "index.html";
}
});

Choose a reason for hiding this comment

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

Really good feature, easy to read. Relevant to the project, so that you can work in your preferred language.

index.en.html Outdated
rel="stylesheet"
/>
<link rel="stylesheet" href="css/style.css" />
<title>Π”Π˜ΠΠ - компания ΠΏΠΎ нСдвиТимости</title>

Choose a reason for hiding this comment

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

Don't forget to add an english title here :)

@KausarShangareeva
Copy link
Author

Good job with this business site, Kausyar, looks really nice! However, it seems like you're missing some requirements:

  • Responsive hamburger menu that's visible only on mobile devices
  • You have divs/spans that look like a form, but I can't see any actual form element which is one of the requirements this week

To add to this, I'm having a hard time checking some of the requirements due to the large amount of files in your repository. Would you mind removing the node_modules (are you using them?) and/or creating a .gitignore file where you write node_modules to make sure they are excluded on GitHub.

Also, there's a little bug when I click the AUD button, then some text appears on top of the existing text. So please also fix this. Image

Im done βœ…πŸ˜€

@HIPPIEKICK
Copy link
Contributor

Good job with this business site, Kausyar, looks really nice! However, it seems like you're missing some requirements:

  • Responsive hamburger menu that's visible only on mobile devices
  • You have divs/spans that look like a form, but I can't see any actual form element which is one of the requirements this week

To add to this, I'm having a hard time checking some of the requirements due to the large amount of files in your repository. Would you mind removing the node_modules (are you using them?) and/or creating a .gitignore file where you write node_modules to make sure they are excluded on GitHub.

Also, there's a little bug when I click the AUD button, then some text appears on top of the existing text. So please also fix this. Image

Can't you see my comment from October 9, @KausarShangareeva ?

@HIPPIEKICK
Copy link
Contributor

HIPPIEKICK commented Nov 27, 2025

  • Still the same bug and still no hamburger menu:
Screenshot 2025-11-27 at 10 15 04
  • Node modules still there, please remove them
  • Form is present βœ…

@KausarShangareeva KausarShangareeva changed the title Finally all Real Estate Website – Technigo Bootcamp Project Dec 1, 2025
@KausarShangareeva KausarShangareeva changed the title Real Estate Website – Technigo Bootcamp Project Real Estate Website – Technigo Bootcamp Project 2025 Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants