Skip to content

Conversation

aiAdrian
Copy link
Contributor

@aiAdrian aiAdrian commented Oct 15, 2025

Description

Fix: We have to send this.signalIsBeingEdited(undefined) to reliably exit pearls view - edit mode, regardless of getVariantIsWritable() or not.

Issues

Checklist

  • This PR contains a description of the changes I'm making
  • I've read the Contribution Guidelines
  • I've added tests for changes or features I've introduced
  • I documented any high-level concepts I'm introducing in documentation/
  • CI is currently green and this is ready for review

…exit Pearls view edit mode, regardless of getVariantIsWritable() or not.
@aiAdrian aiAdrian requested a review from louisgreiner October 15, 2025 08:43
@aiAdrian aiAdrian self-assigned this Oct 15, 2025
@aiAdrian aiAdrian changed the title Fix: Clicking in the background in the Pearl view triggers the signal this.signalIsBeingEdited(undefined) Fix: Clicking in the background in the Pearl view triggers stop editing Oct 15, 2025
Copy link
Contributor

@louisgreiner louisgreiner left a comment

Choose a reason for hiding this comment

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

Nice spot!!

@aiAdrian aiAdrian merged commit 6a9de5c into main Oct 16, 2025
6 checks passed
@aiAdrian aiAdrian deleted the 565-bug-pearlsview-editing-times---clicking-background-to-exit-edit-mode-not-working branch October 16, 2025 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Pearlsview editing times - clicking background to exit edit mode not working

2 participants