You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to scintilla-interest
The Qt ScintillaEditBase has the updateUi signal but doesn't carry the "updated" flags with it. This patch adds this parameter.
Also, there was a "updateUi" signal emitted from ScintillaEditBase::focusInEvent() but as far as I can tell this isn't needed at all...so it has been removed. Any thoughts about this particular line of code and if it is actually needed?
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to scintilla...@googlegroups.com
dail8859:
> The Qt ScintillaEditBase has the updateUi signal but doesn't carry the "updated" flags with it. This patch adds this parameter.
>
> Also, there was a "updateUi" signal emitted from ScintillaEditBase::focusInEvent() but as far as I can tell this isn't needed at all...so it has been removed. Any thoughts about this particular line of code and if it is actually needed?
hg annotate says it is from the original commit so is likely from Jason Haslam / Software Toolworks - they may have more history.
Neil
Jason Haslam
unread,
Jun 23, 2017, 7:42:13 PM6/23/17
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to scintilla...@googlegroups.com
I think that it was just a hack to signal clients to update when a different editor gets focus. It probably doesn’t belong there.