Sitelet https://github.com/inventree/InvenTree/pull/12931
Skip to content

[UI] Fix BOM pricing tooltip - #12931

Merged
SchrodingersGat merged 4 commits into
inventree:masterfrom
SchrodingersGat:z-pricing
Sep 25, 2026
Merged

[UI] Fix BOM pricing tooltip#12931
SchrodingersGat merged 4 commits into
inventree:masterfrom
SchrodingersGat:z-pricing

Conversation

@SchrodingersGat

@SchrodingersGat SchrodingersGat commented Sep 25, 2026

Copy link
Copy Markdown
Member

Closes #12921

@SchrodingersGat SchrodingersGat added this to the 1.6.0 milestone Sep 25, 2026
@SchrodingersGat SchrodingersGat added bug Identifies a bug which needs to be addressed backport Apply this label to a PR to enable auto-backport action User Interface Related to the frontend / User Interface backport-to-1.5.x labels Sep 25, 2026
@netlify

netlify Bot commented Sep 25, 2026 •
edited
Loading

Copy link
Copy Markdown

✅ Deploy Preview for inventree-web-pui-preview ready!

Name Link
🔨 Latest commit 790937d
🔍 Latest deploy log https://app.netlify.com/projects/inventree-web-pui-preview/deploys/6ab61b60a5f77b00096d2e25
😎 Deploy Preview https://deploy-preview-12931--inventree-web-pui-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse		 1 paths audited
Performance: 98 (🟢 up 4 from production)
Accessibility: 81 (🔴 down 1 from production)
Best Practices: 92 (no change from production)
SEO: 78 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

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

@gunstr

gunstr commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

@SchrodingersGat I cannot log in to the preview but have you considered to reduce the tooltip to show only the actual part that you hover over as it used to be instead of the full BoM? I found a single part tooltip more useful for a quick check of which parts are most expensive as it's hard to visually map the colors when the list is long.

@SchrodingersGat

SchrodingersGat commented Sep 25, 2026

Copy link
Copy Markdown
Member Author

@gunstr what you describe is how it works in practice:

image

@SchrodingersGat
SchrodingersGat merged commit 7ece9e8 into inventree:master Sep 25, 2026
49 of 51 checks passed
@SchrodingersGat
SchrodingersGat deleted the z-pricing branch September 25, 2026 23:50
@github-actions github-actions Bot mentioned this pull request Sep 25, 2026
Merged
@github-actions

github-actions Bot commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
✅ 1.5.x

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

SchrodingersGat added a commit that referenced this pull request Sep 26, 2026
@github-actions @SchrodingersGat
[UI] Fix BOM pricing tooltip (#12931) (#12934)
284a670 
* [UI] Fix BOM pricing tooltip

* Add delay for playwright test

(cherry picked from commit 7ece9e8)

Co-authored-by: Oliver <oliver.henry.walters@gmail.com>
@gunstr

gunstr commented Sep 26, 2026

Copy link
Copy Markdown
Contributor

what you describe is how it works in practice:

@SchrodingersGat do you mean that you changed from a full BoM to a single part in the tooltip in this PR?

My current production installation (1.5.2) looks lke this:

Screenshot 2026-09-26 083501

@SchrodingersGat

SchrodingersGat commented Sep 26, 2026

Copy link
Copy Markdown
Member Author

@gunstr can you check against master? It doesn't look this way on my installation (latest / master branch)

@gunstr

gunstr commented Sep 26, 2026

Copy link
Copy Markdown
Contributor

My development master was about two month old and there the tooltip included the full BoM.

Screenshot 2026-09-26 102323

and when I pulled the latest master it changed to only show a single part

Screenshot 2026-09-26 104150

I noticed that in this PR you added tooltipDataSource='segment' - it sounds like that actually is the fix?

The full BoM has been rendered for about a year, maybe there was a default that changed in an underlying library...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

backport Apply this label to a PR to enable auto-backport action backport-to-1.5.x bug Identifies a bug which needs to be addressed User Interface Related to the frontend / User Interface

Projects

None yet

Milestone

1.6.0

Development

Successfully merging this pull request may close these issues.

BOM pricing graph popup has z-index issues

2 participants

@SchrodingersGat @gunstr