Skip to content

textedit: add undo/redo support in textedit - #13394

Open
Montel wants to merge 2 commits into
slint-ui:masterfrom
Montel:add_undo_redo_textedit
Open

Montel wants to merge 2 commits into
slint-ui:masterfrom
Montel:add_undo_redo_textedit

Conversation

@Montel

@Montel Montel commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

cupertino style didn't have enable/readonly support
I sync with other style

@0x6e 0x6e left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The context menu issue needs resolving, I think.

The height of the visible area of the text edit (not including the scrollbar)
</SlintProperty>

## Functions

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This documentation doesn't render correctly (at least in the "On this page" summary).

Working: LineEdit
Broken: TextEdit

Would you be able to fix that as a follow-up please? Otherwise let me know and I can :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I don't understand how my change broke it

@0x6e 0x6e Sep 15, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

You didn't :) I was asking if you could fix it as a follow up/separate PR though, or I will do it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ah ok :) I will fix it this afternoon :)

Comment thread internal/compiler/widgets/common/textedit-base.slint Outdated
Add forwarding functions to the existing TextInput behaviour, and add context menu entries if lineedit
is not enable/read-only.

@0x6e 0x6e left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks a lot!

Comment thread tests/cases/widgets/textedit.slint

@0x6e 0x6e left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Not sure how to un-approve a change.

Comment thread tests/cases/widgets/textedit.slint
Comment thread tests/cases/widgets/textedit.slint
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.

2 participants