Right now the Find Results pane is quite difficult to read for large projects thanks to the fact that tabs and other whitespace are not trimmed from the start of the found lines. What a mess:
It would be trivial and a big improvement to at least trim the leading whitespace, and ideally the results should probably be arranged into a two-column layout (script name / line of code) rather than a simple list.
this + a refresh search button on the window would be amazing
For what it’s worth, Visual Studio is also like ROBLOX when doing a ‘search all’;
Chances are when it was first implemented they wanted it to be something like this, to which it look almost identical.
However, I do agree when a better layout is needed; I would prefer a table, looking something like this;
File | Line | Text
-------------- | ---- | --------------------------------
Test.lua | 1 | print("OMG HI")
GameScript.lua | 1234 | print("This is a debug message")
(Unfortunately, CommonMark does not include tables so I had to manually format that using a code block)
Visual Studio (at least the later versions, not sure what is in your screenshot) also has a built-in widget that also does a “find all” but formats it properly, and you can click the items to jump to the code.
I’m using Visual Studio 2015 (with update 3) in that screenshot; and that’s the only thing I know which allows you to do a find all. Mind telling me where this is? 
Hmm never mind, it appears the one I meant only works for semantic nodes (definitions, variables etc). And you can also click the lines in the output you show anyway, so the benefit is minimal. Getting rid of the leading whitespace and/or having actual columns would be great though, not sure why it’s so crappy even for VS.
we all forgot about the card and it died
idk why it was “toggleable leading whitespace” when just trimming it is what we want and way faster to implement. new card time?
Yeah, there’s no need for a toggle… I can’t think of any reason you would want the indentation.
As far as in comments / disabled scripts, I’m not sure that that’s entirely desirable behavior. Especially disabled scripts: Just because the code isn’t currently running doesn’t mean that it’s not important to know about in a find. Dead code is a source of a lot of bugs if it gets used / reenabled later on.
Yeah that was my own separate idea that I ended up not wanting to push cuz gollygreg’s suggestion was way better lol
Done. No more pesky whitespace in the find widget.