-
Notifications
You must be signed in to change notification settings - Fork 815
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Should TextArea
post TextArea.Changed
message when calling the load_text()
method?
#3660
Comments
TextArea
post TextArea.Changed
message when calling the load_text()
?TextArea
post TextArea.Changed
message when calling the load_text()
method?
Perhaps related to the wider question of when messages should be posted in Textual: |
@TomJGooding, I did not see your issue when searching because I was being too specific in my search term. I agree with what @rodrigogiraoserrao said on your issue. Seems to me that if a widget has a |
Seems to have been fixed in #4256 |
Don't forget to star the repository! Follow @textualizeio for Textual updates. |
I have an app that is tracking changes to a text area so it can update other widgets. If I change the content of the text area using the
load_text()
method shouldn't aTextArea.Changed
message be posted. I understand not doing this during the init but I feel after the init, it may be a nice since the text area does actually change. It wouldn't be too hard to implement a catch that only posts the message if the text content is different from what is currently loaded.I could do a PR if this is something you all feel warrants exploration, and if not, not a big deal as I found a simple work-a-round.
Keep up the great work. Cheers!
The text was updated successfully, but these errors were encountered: