Skip to content

[frontend] When putting number widgets in the middle of a dashboard, it breaks horizontal alignement (#14856)#15003

Open
SarahBocognano wants to merge 5 commits intomasterfrom
issue/14856
Open

[frontend] When putting number widgets in the middle of a dashboard, it breaks horizontal alignement (#14856)#15003
SarahBocognano wants to merge 5 commits intomasterfrom
issue/14856

Conversation

@SarahBocognano
Copy link
Copy Markdown
Member

@SarahBocognano SarahBocognano commented Mar 17, 2026

Proposed changes

  • Wrapped AuditNumber widget component in ContainerWrapper like other Widgets

Related issues

@SarahBocognano SarahBocognano self-assigned this Mar 17, 2026
@SarahBocognano SarahBocognano added the filigran team use to identify PR from the Filigran team label Mar 17, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 17, 2026

Codecov Report

❌ Patch coverage is 2.56410% with 38 lines in your changes missing coverage. Please review.
✅ Project coverage is 33.61%. Comparing base (7904a6e) to head (fee4b3f).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...ti-front/src/components/dashboard/WidgetNumber.tsx 0.00% 10 Missing ⚠️
.../private/components/common/audits/AuditsNumber.jsx 0.00% 8 Missing ⚠️
...common/stix_core_objects/StixCoreObjectsNumber.jsx 0.00% 7 Missing ⚠️
...mon/stix_relationships/StixRelationshipsNumber.jsx 0.00% 7 Missing ⚠️
.../stix_core_objects/PublicStixCoreObjectsNumber.tsx 0.00% 2 Missing ⚠️
...ix_relationships/PublicStixRelationshipsNumber.tsx 0.00% 2 Missing ⚠️
.../opencti-front/src/components/common/card/Card.tsx 50.00% 1 Missing ⚠️
...ti-front/src/components/common/card/CardNumber.tsx 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #15003      +/-   ##
==========================================
- Coverage   33.62%   33.61%   -0.01%     
==========================================
  Files        3189     3190       +1     
  Lines      216746   216750       +4     
  Branches    39957    39952       -5     
==========================================
- Hits        72874    72857      -17     
- Misses     143872   143893      +21     
Flag Coverage Δ
opencti-client-python 45.60% <ø> (ø)
opencti-front 2.94% <2.56%> (+<0.01%) ⬆️
opencti-graphql 70.11% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ancazzz ancazzz requested a review from lndrtrbn March 19, 2026 08:29
Copy link
Copy Markdown
Member

@lndrtrbn lndrtrbn left a comment

Choose a reason for hiding this comment

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

Image

It seems the issue is not fixed for all types of widgets, widgets for entities and relationships still have the issue. Also for the audit widget, the title is duplicated and action buttons not totally aligned

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

Labels

filigran team use to identify PR from the Filigran team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When putting number widgets in the middle of a dashboard, it breaks horizontal alignement

2 participants