I’m working on adding highlighting to code-blocks on lemmy using highlight.js, and am wondering exactly how to implement it. There are many code-themes that could be used Regarding which theme to use there are basically two different directions I could go:

  1. Each lemmy-theme would have to explicitly declare an hljs theme to use.
    • PROS:
      • easy to use, users automatically get an appropriate code-theme for every ui-theme.
      • simple to implement (already done)
    • CONS:
      • theme-makers need to pick an appropriate theme
      • no user customizability, limited number of themes
  2. Users can pick their preferred theme just like they pick a UI theme.
    • PROS:
      • Extreme customization, there are a buttload of themes, and users can pick any one!
    • CONS:
      • Users would need to pick an appropriate theme. It would use the “browser-default” (light/dark) until they pick one, and could look weird and be confusing until then
      • PITA to implement & requires back-end changes (wont see it for a while)

Let me know what you think, or if you have another solution.
I have solution #1 deployed on HeapOverflow.ml right now =]

Here is the PR for those interested: https://github.com/LemmyNet/lemmy-ui/pull/663

  • kevincox
    3·
    2 years ago

    I’m not familiar with how the code works but wouldn’t this be something as simple as

    let codetheme = user_prefs.code_theme
  .or(user_prefs.ui_theme.recommended_code_theme)
  .unwrap_or(DEFAULT_CODE_THEME)
    • ThannOP
      1·
      2 years ago

      It could be, but if I put the mapping of UI-theme to code theme in JS then custom themes would require custom code on the frontend. My simple implementation for #1 only involves adding an ‘@import myhljstheme.css’ statement to the themes css. This makes themeing easier, but css is blissfully unaware of what happens in JavaScript land.

      • kevincox
        2·
        2 years ago

        Ah, I see. Yeah, if you just compile it into the theme it is slightly simpler. But yeah, I think ideally the frontend could pull UI Theme + Code Theme separately to allow flexibility.