Skip to content

fix(UX): Misc#462

Merged
surajshetty3416 merged 9 commits intodevelopfrom
fix-variable-manager
Jan 12, 2026
Merged

fix(UX): Misc#462
surajshetty3416 merged 9 commits intodevelopfrom
fix-variable-manager

Conversation

@surajshetty3416
Copy link
Member

@surajshetty3416 surajshetty3416 commented Jan 11, 2026

- and update frappe-ui
So that it does not always stays on top when modal appears because modal gets appended to the body and it should always be on top and not the popup

#456 this fix worked but it completely hides popup till the modal is open which creates unnecessary UI flashes and causes confusion
@surajshetty3416 surajshetty3416 changed the title fix: Misc fix(UX): Misc Jan 11, 2026
@surajshetty3416 surajshetty3416 marked this pull request as ready for review January 11, 2026 17:31
@codecov
Copy link

codecov bot commented Jan 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 49.26%. Comparing base (e09b830) to head (67c48fc).
⚠️ Report is 48 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #462   +/-   ##
========================================
  Coverage    49.26%   49.26%           
========================================
  Files           28       28           
  Lines         2176     2176           
========================================
  Hits          1072     1072           
  Misses        1104     1104           

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

@surajshetty3416 surajshetty3416 merged commit 79cf494 into develop Jan 12, 2026
5 of 6 checks passed
@github-actions
Copy link

🎉 This PR is included in version 1.22.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant