Quoting spoiler text doesn't include spoiler tags in the quote

AI-generated summary

Quoting Spoiler Text Issue

A bug has been reported by seanblue where quoting text that includes spoiler tags does not include the spoiler tags in the quote. Falco explained that this is due to the HTML->Markdown conversion library used in the select text and quote feature.

seanblue mentioned that this functionality used to work and was even improved in June of this year. Although a workaround is available by using the quote full post feature, seanblue inquired about the possibility of a fix. codinghorror acknowledged the issue and suggested making necessary edits in quotes as a temporary solution.

The issue has now been confirmed and a fix is underway, as announced by zogstrip with a link to the GitHub pull request: https://github.com/discourse/discourse/pull/27232.

There’s currently an issue where highlighting text that includes spoiler text doesn’t actually put the spoiler tags in the quote. This definitely used to work, but I’m not sure when it broke.

For example, if you highlight this entire sentence, the spoiler part isn’t behind spoiler tags in the quote.

2 Likes

1 Like

As far as I know the select text and quote feature will do a best effort work trying to preserve formatting using our HTML->Markdown conversion library.

If you use the quote full post (first button in the composer) we go to the server and ask for the raw markdown, and then the quote contains all the original formatting.

1 Like

Support for quoting spoiler text was added back in August 2018 and it was improved in June of this year (even if incidentally) as referenced here. So while I understand that I can do the full post quote as a workaround, it remains that this is functionality that used to work and is now broken.

3 Likes

Any chance this will be fixed?

1 Like

Probably eventually, for now you can make the necessary edits in quotes, so there is a workaround.

1 Like

Okay. Just wanted some kind of confirmation, so thanks.

2 Likes

I just realized this is still broken. :cry:

1 Like

Thanks @seanblue for the report and your (extreme) patience :sweat_smile:

It will be fixed by :point_down:

https://github.com/discourse/discourse/pull/27232

(cc @lindsey for visibility)

3 Likes

This topic was automatically closed after 3 days. New replies are no longer allowed.