Skip to content

Manoj - Add inventory navigation icons, cross-page links, and BM Dashboard dropdown under Other Links#5306

Open
ManojPuttaswamy wants to merge 15 commits into
developmentfrom
Manoj_BMDashboard
Open

Manoj - Add inventory navigation icons, cross-page links, and BM Dashboard dropdown under Other Links#5306
ManojPuttaswamy wants to merge 15 commits into
developmentfrom
Manoj_BMDashboard

Conversation

@ManojPuttaswamy
Copy link
Copy Markdown
Contributor

@ManojPuttaswamy ManojPuttaswamy commented May 30, 2026

Description

Screenshot 2026-03-27 at 10 51 01 AM

Related PRS (if any):

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 /bmdashboard
  6. verify bmdashboard header persistence under other links
  7. verify this new feature “All Inventory Types” under the BM projects dropdown
  8. verify navigation of each categories in “All Inventory Types” page
  9. verify back to all inventory type functionality from each category
  10. verify dropdown structure of BM Projects
  11. verify icons added for inventory types
  12. verify functionalities in dark mode

Screenshots or videos of changes:

REC-20260530200247.mp4

Notes:

@netlify
Copy link
Copy Markdown

netlify Bot commented May 30, 2026

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit 323a310
🔍 Latest deploy log https://app.netlify.com/projects/highestgoodnetwork-dev/deploys/6a22095bc2fe5e0008ff25f5
😎 Deploy Preview https://deploy-preview-5306--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@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 May 30, 2026
Copy link
Copy Markdown

@DeepighaJ DeepighaJ left a comment

Choose a reason for hiding this comment

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

  • Verified navigation to BM Dashboard through Other Links and confirmed the BM Dashboard header remains persistent after navigation as expected.
  • Verified the new "All Inventory Types" option is available under the BM Projects dropdown and navigates correctly.
  • Verified all inventory categories (Materials, Consumables, Equipments, Reusables, Tools, and Unit of Measurement) can be expanded and collapsed successfully.
  • Verified navigation from All Inventory Types to individual category pages and confirmed category data is displayed correctly.
  • Verified Back to All Inventory Types functionality works as expected from category pages.
  • Verified the updated BM Projects dropdown structure and inventory type icons are displayed correctly.

Issue Found
Dark Mode UI Inconsistency: When navigating back from category pages (e.g., Materials, Consumables) to All Inventory Types using the "All Inventory Types" button or icons, some table elements retain light-theme styling instead of adopting dark-mode colors. This results in reduced readability and an inconsistent user experience in Dark Mode. (Refer last screenshot.)
Image
Image
Image
Image
Image
Image
Image
Image
Image

Copy link
Copy Markdown

@HemanthNidamanuru HemanthNidamanuru left a comment

Choose a reason for hiding this comment

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

Hi Manoj,

I tested this locally following the steps provided and found 2 issues:

Unit of Measurement is missing from the BM Projects dropdown.
Dark mode - The BM Projects dropdown has a slight misalignment in dark mode.

PR.5306.mp4

Copy link
Copy Markdown

@amaresh2001 amaresh2001 left a comment

Choose a reason for hiding this comment

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

Tested locally on Manoj_BMDashboard.

Testing:

  • "BM Dashboard" is correctly listed under the Other Links dropdown
  • "All Inventory Types" appears in the BM Projects dropdown and routes correctly to /bmdashboard/inventorytypes
  • Dropdown closes after selecting an option
  • Categories (Materials, Consumables, Equipment, Reusables, Tools) are clickable navigation links with icons
  • "← All Inventory Types" back link is present on each category page

Issues:

  • Unit of Measurement is not a clickable link; all other categories on the Inventory Types page are clickable navigation links.
  • Dark mode inconsistency on navigation back when in dark mode and navigating from a category page back to /bmdashboard/inventorytypes, the page body renders with a white/light background while the navbar remains dark. The page does not fully apply dark mode styling on return navigation.
Image Image Image Image Image Image Image Image

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 4, 2026

@ManojPuttaswamy
Copy link
Copy Markdown
Contributor Author

  • Verified navigation to BM Dashboard through Other Links and confirmed the BM Dashboard header remains persistent after navigation as expected.
  • Verified the new "All Inventory Types" option is available under the BM Projects dropdown and navigates correctly.
  • Verified all inventory categories (Materials, Consumables, Equipments, Reusables, Tools, and Unit of Measurement) can be expanded and collapsed successfully.
  • Verified navigation from All Inventory Types to individual category pages and confirmed category data is displayed correctly.
  • Verified Back to All Inventory Types functionality works as expected from category pages.
  • Verified the updated BM Projects dropdown structure and inventory type icons are displayed correctly.

Issue Found Dark Mode UI Inconsistency: When navigating back from category pages (e.g., Materials, Consumables) to All Inventory Types using the "All Inventory Types" button or icons, some table elements retain light-theme styling instead of adopting dark-mode colors. This results in reduced readability and an inconsistent user experience in Dark Mode. (Refer last screenshot.) Image Image Image Image Image Image Image Image Image

Hi, Dark mode inconsistency has been fixed. Please re review it and Thankyou.

@ManojPuttaswamy
Copy link
Copy Markdown
Contributor Author

Tested locally on Manoj_BMDashboard.

Testing:

  • "BM Dashboard" is correctly listed under the Other Links dropdown
  • "All Inventory Types" appears in the BM Projects dropdown and routes correctly to /bmdashboard/inventorytypes
  • Dropdown closes after selecting an option
  • Categories (Materials, Consumables, Equipment, Reusables, Tools) are clickable navigation links with icons
  • "← All Inventory Types" back link is present on each category page

Issues:

  • Unit of Measurement is not a clickable link; all other categories on the Inventory Types page are clickable navigation links.
  • Dark mode inconsistency on navigation back when in dark mode and navigating from a category page back to /bmdashboard/inventorytypes, the page body renders with a white/light background while the navbar remains dark. The page does not fully apply dark mode styling on return navigation.

Image Image Image Image Image Image Image Image

Hi, the Issue 01 that you have mentioned, I believe Unit of Measurement is not supposed to be a clickable link. And, the second issue has been resolved. Please re review it. Thank you

@ManojPuttaswamy
Copy link
Copy Markdown
Contributor Author

Hi Manoj,

I tested this locally following the steps provided and found 2 issues:

Unit of Measurement is missing from the BM Projects dropdown. Dark mode - The BM Projects dropdown has a slight misalignment in dark mode.

PR.5306.mp4

Hi, The first issue, I believe that the unit of measurement is not a category and hence there is no navigation to it. The second issue has been resolved. Please re review it. Thankyou

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.

5 participants