search

plibither8 / refined-hacker-news

✨ Hacker News, but refined — Interface tweaks and features to make the HN experience better
MIT License
807 stars 34 forks source link

Feature request: convert text surrounded by backticks to monospace. #74

Closed sjuxax closed 4 years ago

sjuxax commented 4 years ago

Most developer-oriented sites will take text surrounded by backticks and make it an inline-monospace font with some highlighting, like so. I've always hated that HN doesn't have this and sometimes follow the convention anyway (this comment, for example) in hopes that it would one day become part of the codebase. It'd be awesome if this extension could close that gap. :)

plibither8 commented 4 years ago

Hi Jeff, thanks for the request! Seems to be a very reasonable request, and one that would be unanimously appreciated. I'll get to work on as soon as I get some time.

plibither8 commented 4 years ago

Done! This might need some refinement in the future to cover edge-cases, but I guess it works well enough. I'll be pushing the changes to the marketplaces right now, but the Chrome extension might take some time to update.

Thanks once again!

chris-morgan commented 4 years ago

👎 I strongly believe that this should not be enabled by default, because it changes the meaning of text not uncommonly and is changing the meaning of Hacker News formatting. Immediately after #75 landed, I was talking about reStructuredText on a thread, which uses backticks in some places, and this butchered my comment.

It would be OK for Hacker News to introduce something like this—people will then work around it or not as the case may be—but so long as the rest of Hacker News is not treating backticks specially, it’s very bad for an extension to default to mangling content.

I suggest that this be disabled by default.

plibither8 commented 4 years ago

Hi @chris-morgan, thanks for the comment. While I do understand your concern here, I won't be able to disable default: currently I have not worked up a mechanism to disable features by default. Once I do, I will surely look into this again.