pajlada

Results 306 comments of pajlada

Annotations definitely looks like a suitable alternative, however I believe the 10 annotation limit is correct, and losing the Suggestions part is unfortunate. For 2), I would also prefer using...

@frankban I've gone ahead and pushed missing tests for this functionality to maintain the 100% test coverage of `checker.go`, let me know if you prefer this to be rebased or...

Closing this one out since it's too generic, feel free to open a new issue if you find a pattern to the crashes or if you're able to attach a...

Would also love a video showing off the functionality

Super interesting feature! Would love to get this merged in. Haven't had a time to review the actual code yet but thank you for working on it!

Unable to crash this with popups https://user-images.githubusercontent.com/962989/184422432-175be597-d7a1-4aa5-873b-c62c9d3bc5c0.mp4

I haven't added a test for the ` // Emote coords are out of range from the modified character positions` error case because I can't figure out a fake message...

Issue with tests failing has to do with the `MockApplication` class symbols being merged/broken somehow - the fix is to put it in an anonymous namespace, making sure no other...

Tests should work now :-)

@TroyKomodo Merge the latest main branch into your branch