Move property header to sidebar with adjusted column widths#93
Move property header to sidebar with adjusted column widths#93ewebtechsuk merged 2 commits intomainfrom
Conversation
There was a problem hiding this comment.
Codex Review: Here are some suggestions.
Reply with @codex fix comments to fix any unresolved comments.
About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".
| } ?> | ||
| </div> | ||
|
|
||
| <div class="col-12 col-lg-4 right-column"> | ||
| <div class="property-header mt-4"> | ||
| <div class="d-flex flex-column flex-md-row justify-content-between align-items-md-center gap-2"> | ||
| <div> | ||
| <h1 class="property-title mb-1"><?=htmlspecialchars($details->displayAddress)?></h1> |
There was a problem hiding this comment.
[P1] Preserve property header above main content on small screens
The commit removes the full‑width header that previously preceded the details row and leaves the only remaining header inside the right sidebar (col-12 col-lg-4 right-column). Because the left column is rendered first in the DOM and the new CSS only applies widths for min-width: 992px, on viewports below 992 px the columns stack and the header (title, price, CTAs) now appears after the photo gallery, features and description. This regresses the mobile layout by pushing the most important information and actions to the bottom of the page. Consider keeping a top header for small screens or reordering the columns with Bootstrap’s order utilities so the header remains first when stacked.
Useful? React with 👍 / 👎.
Summary
mt-4Testing
npm testhttps://chatgpt.com/codex/tasks/task_e_68bf7cd41bbc832e8fbd032f25c8c272