We currently trigger a select command update whenever the selection changes in any text field. However our select commandupdaters only ever care about the selection in the focused field. All other events are therefore pointless. This applies to undo command updates too.
Created attachment 280807 [details] [diff] [review] Like so It's not an issue for undo, because we're not an editor listener unless we're focused.
10 years ago
Comment on attachment 280807 [details] [diff] [review] Like so I assume there's no other easy way to check whether we have focus?
> I assume there's no other easy way to check whether we have focus? Well... we could GetContentState(mFrame->GetContent()) on the ESM and see whether it has the FOCUSED state. I think that should be equivalent, though slower. Memory usage will be the same, since I'm adding a 4th PRPackedBool here. I can do the content state thing if you'd prefer, though.
I think I'd prefer that ... extra state that has to be maintained adds a little more complexity.
Created attachment 280949 [details] [diff] [review] With that approach