Skip to content

fix(ui): Fix update banner hidden behind canvas controls - #12527

Merged
Cristhianzl merged 1 commit into
release-1.9.0from
cz/fix-update-warning
Apr 7, 2026
Merged

fix(ui): Fix update banner hidden behind canvas controls#12527
Cristhianzl merged 1 commit into
release-1.9.0from
cz/fix-update-warning

Conversation

@Cristhianzl

Copy link
Copy Markdown
Member

OBJECTIVE: Reposition the component update banner so it appears above the canvas controls instead of being obscured behind them.

CHANGES:

  • Change UpdateAllComponents bottom positioning from bottom-2 to bottom-16 to match FlowBuildingComponent placement
image

@coderabbitai

coderabbitai Bot commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 64fae271-d692-4814-8485-d8eb188c9b2f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cz/fix-update-warning

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added the bug Something isn't working label Apr 6, 2026
@codecov

codecov Bot commented Apr 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.03%. Comparing base (2f6400d) to head (04d04c7).
⚠️ Report is 11 commits behind head on release-1.9.0.

❌ Your project status has failed because the head coverage (49.06%) is below the target coverage (60.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                @@
##           release-1.9.0   #12527      +/-   ##
=================================================
+ Coverage          49.89%   50.03%   +0.14%     
=================================================
  Files               1930     1930              
  Lines             171894   172595     +701     
  Branches           25319    25326       +7     
=================================================
+ Hits               85762    86356     +594     
- Misses             85086    85193     +107     
  Partials            1046     1046              
Flag Coverage Δ
backend 55.93% <ø> (ø)
frontend 48.89% <100.00%> (+0.21%) ⬆️
lfx 49.06% <ø> (ø)

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

Files with missing lines Coverage Δ
.../FlowPage/components/UpdateAllComponents/index.tsx 95.83% <100.00%> (ø)

... and 104 files with indirect coverage changes

🚀 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.

@Cristhianzl Cristhianzl self-assigned this Apr 6, 2026
@github-actions

github-actions Bot commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

Frontend Unit Test Coverage Report

Coverage Summary

Lines Statements Branches Functions
Coverage: 29%
29.69% (31319/105481) 63.69% (3976/6242) 31.04% (731/2355)

Unit Test Results

Tests Skipped Failures Errors Time
3104 0 💤 0 ❌ 0 🔥 5m 14s ⏱️

@github-actions github-actions Bot added the lgtm This PR has been approved by a maintainer label Apr 6, 2026
@Cristhianzl
Cristhianzl merged commit 74f17b4 into release-1.9.0 Apr 7, 2026
97 of 98 checks passed
@Cristhianzl
Cristhianzl deleted the cz/fix-update-warning branch April 7, 2026 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lgtm This PR has been approved by a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants