Support for <pre><code> basic pastebins about linkembedder HOT 5 OPEN

Without changing the selectors, LE could support <pre><code> by changing

to

$self->{paste} = $tmp->all_text;

or by adding something like

if (my $code = $tmp->at('code')) { $tmp = $code }

above it. I'd be happy to put together a PR for either solution if you'd accept them @jhthorsen . Just let me know.

(adding a selector for body pre > code would be fine by me too remember).

from linkembedder.

I've been giving this some more thought, and I think we should "break" LinkEmbedder by adding the fix you suggest. By "breaking" I mean that it will probably find some non-pastebin sites and render them as a pastebin. I'm thinking about blogs with code examples and such. I do think breaking it, and then later on adding better tests would be better than holding this change back.

Feel free to open a PR under these conditions (if possible)

• Add a test for a working site
• Document in the code what the selector is intended to match, so we don't accidentally break the change you want.

About the "document in the code" point: I should've done that myself, but I haven't so now we're in this situation 😞

from linkembedder.

Seems like all_text has already been implemented, so I'm going to close this issue.

Let me know if I forgot something @jberger.

from linkembedder.

...or just open a PR with a sloppy "pre" lookup, and I'll merge it.

from linkembedder.

I'm reopening this, since https://svelte.dev/docs is a case where it gets expanded to a <pre> tag when it shouldn't imo. Might also discuss if https://svelte.dev/docs#4_Prefix_stores_with_$_to_access_their_values should get rendered as a code example.

Could this be fixed simply by checking if there's a "bunch" of <pre> tags? Where bunch is more than 1 or 2..?

And even this issue: Looks like LinkEmbedder finds the code link in #24 (comment) - Thought that's probably Github.pm specific.

from linkembedder.