search

chaosparrot / talon_hud

Unofficial Talon Head Up Display scripts
68 stars 29 forks source link

Fixed issue where cursor tracker sometimes wouldn't show. #14

Closed wolfmanstout closed 2 years ago

wolfmanstout commented 2 years ago

Happy to test this for couple days if you have any concerns, but this seems to fix an issue I was having where the cursor tracker I created would intermittently not show when switching it off and on again (I tie it to a mode).

chaosparrot commented 2 years ago

Hmm, that is interesting. Seems like this is just doing a copy of the list right? If so, it seems good to me.

wolfmanstout commented 2 years ago

That's right. I was seeing instances where update_icons wouldn't trigger soft_enable because self.cursor_icons was already equal to the new cursor_icons due to reuse of the same list object.

chaosparrot commented 2 years ago

Cool, seems logical, I'll merge it. Thanks for making a PR :)