Fix comparisons across the project

This improves performance because way less string objects are created, and also fix a few globalization warnings.
Just wondering if swapping the terms of the boolean expression would be more efficient,
`(isItalic && text.Substring(i).StartsWith("</i>", StringComparison.OrdinalIgnoreCase))`