Link to rendered GitHub files should include `?plain=1` about gitweblinks HOT 5 CLOSED

It looks like f1ec882 adds ?plain=1 to every GitHub URL, even those that default to plain rendering.

The downside is that it looks like ?plain=1 suppresses embedded code snippets in GitHub comments. See the following URLs and how they render:

https://github.com/reduckted/vscode-gitweblinks/blob/1384c1e5ee29d3b57ae3fbb31fd9140f3e7fadcc/shared/handlers/bitbucket-server.json#L6-L7

https://github.com/reduckted/vscode-gitweblinks/blob/1384c1e5ee29d3b57ae3fbb31fd9140f3e7fadcc/shared/handlers/bitbucket-server.json#L6-L7

https://github.com/reduckted/vscode-gitweblinks/blob/1384c1e5ee29d3b57ae3fbb31fd9140f3e7fadcc/shared/handlers/bitbucket-server.json?plain=1#L6-L7

https://github.com/reduckted/vscode-gitweblinks/blob/1384c1e5ee29d3b57ae3fbb31fd9140f3e7fadcc/shared/handlers/bitbucket-server.json?plain=1#L6-L7

https://github.com/reduckted/vscode-gitweblinks/blob/1384c1e5ee29d3b57ae3fbb31fd9140f3e7fadcc/README.md?plain=1#L14-L16

https://github.com/reduckted/vscode-gitweblinks/blob/1384c1e5ee29d3b57ae3fbb31fd9140f3e7fadcc/README.md?plain=1#L14-L16

IMO this is a GitHub bug. ?plain=1 should not suppress embedded code snippets (I emailed support). But since it does, I wonder if ?plain=1 should only be added to files ending with .md or .markdown?

from gitweblinks.

@reduckted the plain is a pain because I need to manually remove it to get the previews as @dhimmel shows. Any chance this could be made configurable, or @dhimmel's suggestion implemented?

from gitweblinks.

Thanks for the info. I've added this v2.5.1 which is available in the marketplace now.

from gitweblinks.

Thanks! Noting f1ec882 as the relevant commit.

from gitweblinks.

I've improved this in 4b1b35e. The ?plain=1 is only added to markdown files (.md or .markdown).

Version 2.5.3 should be available on the marketplace shortly.

from gitweblinks.