Skip to content

Conversation

@SwathiAngadi
Copy link

LB Dashboard messaging UI was rendering the blank page due to CSS issues.

Related PRS (if any):

NA

Main changes explained:

  • Fixed the CSS changes for the whole page.
  • Applied Dark Mode theme for the UI.
  • Made sure page is responsive too and adopted for dark mode theme.
  • Applied standard color code present in Figma for cleaner and presentable UI.
  • Removed redundant css changes from the modular.css file.
  • Removed multiple declaration of routes for this page.

How to test:

  1. check into current branch
  2. do npm install and ... to run this PR locally
  3. Clear site data/cache
  4. Login as Admin user.
  5. Navigate to /lbdashboard/messaging
  6. Check for Visual Appearance.
  7. Check for Dark mode compatibility.
  8. Check for responsiveness.

Screenshots or videos of changes:

Screenshot 2025-12-31 153313 Screenshot 2025-12-31 153336 Screenshot 2025-12-31 153500 Screenshot 2025-12-31 153525 Screenshot 2025-12-31 154209 Screenshot 2025-12-31 154222

Note:

LBDasboard components misses the header section. This issue is taken as part of other PR.
This PR concentrates on the visual appearance of the page.
Raised issue for " When user sends message, in the body multiple messages are triggered."

@netlify
Copy link

netlify bot commented Dec 31, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit af7a6c8
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6955b8b1cf12f80008837c78
😎 Deploy Preview https://deploy-preview-4652--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

Quality Gate Failed Quality Gate failed

Failed conditions
14.3% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

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.

2 participants