Skip to content

Fix: pasted triggers losing content#8303

Merged
vadi2 merged 5 commits intoMudlet:developmentfrom
sammerpetria:fixcopy
Oct 12, 2025
Merged

Fix: pasted triggers losing content#8303
vadi2 merged 5 commits intoMudlet:developmentfrom
sammerpetria:fixcopy

Conversation

@sammerpetria
Copy link
Copy Markdown
Contributor

@sammerpetria sammerpetria commented Oct 5, 2025

Brief overview of PR changes/additions

Adjusted trigger editor selection handler to prevent clearing the newly pasted trigger before it gets saved
Fix: #8249 pasted trigger via loose content
this was originated after a litle collateral after #8056

Motivation for adding to Mudlet

Fix 8249

Other info (issues closed, discussion etc)

this could be related to issues 8259 and 8293. as first step to solve (but this PR dont solve thoise)

Signed-off-by: SammerPetria <sammerpetria@gmail.com>
@sammerpetria sammerpetria requested a review from a team as a code owner October 5, 2025 12:12
Copy link
Copy Markdown
Member

@vadi2 vadi2 left a comment

Choose a reason for hiding this comment

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

I can confirm #8249 is fixed, but #8259 still happens for me.

@Kebap can you test this as well?

@sammerpetria
Copy link
Copy Markdown
Contributor Author

I can confirm #8249 is fixed, but #8259 still happens for me.

@Kebap can you test this as well?

You r right, only #8429 solved here.
my bad i miss understand the 8259 issue and think solved.
I think are related, but this PR dont solve 8259. I can try in a another PR after this merged, as first step i think.

@vadi2 vadi2 changed the title Fix: pasted trigger loose content #8249 and #8259 Fix: pasted triggers losing content Oct 12, 2025
@vadi2 vadi2 merged commit 815ceec into Mudlet:development Oct 12, 2025
13 checks passed
@Kebap
Copy link
Copy Markdown
Contributor

Kebap commented Oct 16, 2025

@Kebap can you test this as well?

Sorry was away a few days

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pasted triggers loose their contents

3 participants