Skip to content

feat: Preserve inactive content for feature notifications - #5158

Draft
georgylobko wants to merge 1 commit into
mainfrom
feat/preserve-inactive-content-feature-notifications
Draft

georgylobko wants to merge 1 commit into
mainfrom
feat/preserve-inactive-content-feature-notifications

Conversation

@georgylobko

Copy link
Copy Markdown
Member

Description

Add a support for preserseInactiveContent for local drawers, same as for global drawers, to enable feature notifications preserve their runtime content without unmounting when closed.

Related links, issue #, if available: n/a

How has this been tested?

Review checklist

The following items are to be evaluated by the author(s) and the reviewer(s).

Correctness

  • Changes include appropriate documentation updates.
  • Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
  • Changes do not include unsupported browser features, see CONTRIBUTING.md.
  • Changes were manually tested for accessibility, see accessibility guidelines.

Security

Testing

  • Changes are covered with new/existing unit tests?
  • Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@codecov

codecov Bot commented Oct 19, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.77419% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 97.72%. Comparing base (af861d1) to head (4f2003d).

Files with missing lines Patch % Lines
...ut/visual-refresh-toolbar/drawer/local-drawers.tsx 91.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5158      +/-   ##
==========================================
- Coverage   97.72%   97.72%   -0.01%     
==========================================
  Files         995      996       +1     
  Lines       32129    32139      +10     
  Branches    11851    11853       +2     
==========================================
+ Hits        31399    31408       +9     
- Misses        684      685       +1     
  Partials       46       46              

☔ View full report in Codecov by Harness.
📢 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.

This branch had an error being deployed

1 failed and 2 active deployments
allure-report — 4f2003d0 Deployed Oct 19, 2026 by georgylobko via Visual regression / Deploy Allure Report / deploy #2841
dev-pages-react16 — 4f2003d0 Deployed Oct 19, 2026 by georgylobko via deploy (React 16) / deploy #2841
dev-pages-react18 — 4f2003d0 Deployed Oct 19, 2026 by georgylobko via deploy (React 18) / deploy #2841
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant