rbolsius

Results 7 comments of rbolsius

I had no problem following the code. It is very clear and well organized. Wish I could say the same for all the code I read. The main reason I...

I think a simple fix is to handle empty strings as a special-case in phi-search--make-overlays-for. ``` (defun phi-search--make-overlays-for (query &optional unlimited) ... (cond ((string= "" query) (phi-search--message "too short")) ((not...

Here is a demonstration of the table formatting before the fix: ![table-formatting-test-before](https://user-images.githubusercontent.com/2106964/171906536-c1c1533b-ce79-4947-beaa-1e6c1458fd55.gif) And after the fix: ![table-formatting-test-after](https://user-images.githubusercontent.com/2106964/171906567-141a8e77-3950-44f8-9fd0-b8f3f409e750.gif) Here is the original Markdown used as the test case: ``` # Table...

This fix in this pull request would be very welcomed. I have INI files on platforms with case-sensitive file systems that do not work with this extension because the file...

It would be very appreciated if either PR #1 or #2 could be merged. It will help those of us who use VS Code on hosts with case-sensitive file systems...

I did a bit more digging and found that this issue appears to be an incompatibility between this extension and the [Overtype] extension. When either one of these extensions is...

Since it is not clear which extension is at fault, I have also created the following issue against the Overtype extension. https://github.com/DrMerfy/vscode-overtype/issues/38