Skip to content

Durga Venkata Praveen Fix: Header responsive layout and CSS module conversion#4235

Merged
one-community merged 2 commits intodevelopmentfrom
DurgaVenkataPraveen-HeaderAlignmentFixing
Nov 12, 2025
Merged

Durga Venkata Praveen Fix: Header responsive layout and CSS module conversion#4235
one-community merged 2 commits intodevelopmentfrom
DurgaVenkataPraveen-HeaderAlignmentFixing

Conversation

@boppanapraveen
Copy link
Copy Markdown
Contributor

@boppanapraveen boppanapraveen commented Oct 18, 2025

Description

image

Also includes:
image

From this closed PR: #4013

Related PRS (if any):

No Related PR's

Main changes explained:

  • Flex Wrapping for Responsiveness

  • Enabled flex-wrap: wrap for .timer-message-section and .navbar so header content can wrap onto new lines when the screen is narrow.

  • Header Width
    Changed .header-wrapper width from clamp(100vw, 0.1rem + 1vw, 100%) to width: 100% for better scaling and to prevent overflow.
    Responsive Margins and Padding

  • Added media queries to reduce .headerCard margins and .card-content paddings on screens smaller than 900px.
    Header Message Responsiveness

  • In screens below 1050px, .timer-message-section switches to column layout and centers its content.
    .owner-message (the header message) is centered, uses italic font, wraps text, and is stacked below the timer, matching your desired mobile/tablet layout.

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. log as admin user
  5. go to dashboard→ Tasks→ task→…
  6. verify function “A” (feel free to include screenshot here)
  7. verify this new feature works in dark mode

Screenshots or videos of changes:

image image image image

Note:

Include the information the reviewers need to know.

@netlify
Copy link
Copy Markdown

netlify Bot commented Oct 18, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 947f917
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/68f44c14aba24600084ca9db
😎 Deploy Preview https://deploy-preview-4235--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@sonarqubecloud
Copy link
Copy Markdown

@hemanthvenkat
Copy link
Copy Markdown

Verified the Header: responsive layout behaves correctly across breakpoints (logo/nav alignment, menu collapse, no layout shifts), with light/dark parity.
CSS Module scoping works as intended—no style leakage, predictable class names, and consistent spacing/typography.
Overall, visuals and interactions meet spec with no console errors.

image image

Copy link
Copy Markdown
Contributor

@akshith312 akshith312 left a comment

Choose a reason for hiding this comment

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

Tested PR locally.
The layout is responsive and works in both light and dark modes for various screen sizes.

image image image image

@beblicarl
Copy link
Copy Markdown
Contributor

This responsive fix works perfectly

image image image

@beblicarl beblicarl self-requested a review November 7, 2025 11:49
@one-community one-community added the High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible label Nov 10, 2025
Copy link
Copy Markdown

@Anusha-Gali Anusha-Gali left a comment

Choose a reason for hiding this comment

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

Hi Praveen,

I have tested your PR locally and the header responsiveness works as per the functionality mentioned.

Screenshot 2025-11-11 at 3 19 35 PM Screenshot 2025-11-11 at 3 24 52 PM Screenshot 2025-11-11 at 3 19 16 PM Screenshot 2025-11-11 at 3 29 50 PM

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit 71f7e10 into development Nov 12, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

High Priority - Please Review First This is an important PR we'd like to get merged as soon as possible

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants