Skip to content

fix(toast): position when closed - #2600

Merged
skjnldsv merged 1 commit into
mainfrom
pulsejet/nav-closed
Sep 8, 2026
Merged

skjnldsv merged 1 commit into
mainfrom
pulsejet/nav-closed

Conversation

@pulsejet

@pulsejet pulsejet commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

It adds --closed not --close so the toast has wrong position on mobile.

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

It adds --closed not --close so the toast has wrong position on mobile.

Signed-off-by: Varun Patil <varunpatil@ucla.edu>
@pulsejet
pulsejet force-pushed the pulsejet/nav-closed branch from 6a6795d to d6cb246 Compare September 8, 2026 02:24
@pulsejet
pulsejet requested a review from skjnldsv September 8, 2026 02:24
@codecov

codecov Bot commented Sep 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.86%. Comparing base (70cb8df) to head (d6cb246).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2600   +/-   ##
=======================================
  Coverage   59.86%   59.86%           
=======================================
  Files          15       15           
  Lines         461      461           
  Branches       96       96           
=======================================
  Hits          276      276           
  Misses        164      164           
  Partials       21       21           

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

@skjnldsv
skjnldsv merged commit 829c8b3 into main Sep 8, 2026
15 checks passed
@skjnldsv
skjnldsv deleted the pulsejet/nav-closed branch September 8, 2026 06:54
@skjnldsv skjnldsv added bug Something isn't working 3. to review labels Sep 8, 2026
@pulsejet

Copy link
Copy Markdown
Contributor Author

@skjnldsv what's the release cadence? Need a release for this patch

pulsejet added a commit to pulsejet/memories that referenced this pull request Sep 13, 2026
Small NcDialog confirmations went fullscreen on mobile and toasts were offset when navigation is closed. Temporary local overrides until the upstream fixes land.

nextcloud-libraries/nextcloud-vue#8956 nextcloud-libraries/nextcloud-dialogs#2600
Signed-off-by: Varun Patil <radialapps@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants