markdown-viewer icon indicating copy to clipboard operation
markdown-viewer copied to clipboard

Support upper case MD extension

Open danintel opened this issue 6 years ago • 2 comments
trafficstars

The RE \.(?:markdown|mdown|mkdn|md|mkd|mdwn|mdtxt|mdtext|text)(?:#.*|\?.*)?$ does not support .MD. I modified it as follows and it worked: \.(?:markdown|mdown|mkdn|md|MD|mkd|mdwn|mdtxt|mdtext|text)(?:#.*|\?.*)?$

danintel avatar Sep 06 '19 17:09 danintel

That's a good point. I wonder what is the performance difference between case sensitive and case insensitive regex in this case. I might have to make a few tests.

simov avatar Sep 06 '19 18:09 simov

I don't know. An alternative is to copy the string and convert to lower case--for comparison purposes only.

danintel avatar Sep 06 '19 18:09 danintel