this post was submitted on 19 Aug 2024
55 points (98.2% liked)

Programming

16995 readers
252 users here now

Welcome to the main community in programming.dev! Feel free to post anything relating to programming here!

Cross posting is strongly encouraged in the instance. If you feel your post or another person's post makes sense in another community cross post into it.

Hope you enjoy the instance!

Rules

Rules

  • Follow the programming.dev instance rules
  • Keep content related to programming in some way
  • If you're posting long videos try to add in some form of tldr for those who don't want to watch videos

Wormhole

Follow the wormhole through a path of communities [email protected]



founded 1 year ago
MODERATORS
 

Hey all, I made a Firefox extension (signed by Mozilla) specifically to add Show/Hide Child Comments functionality similar to how RES had it (where the parent comment is still visible).

It's not very useful, but I could use some feedback on tightening up the Javascript. I'm not a JS beginner, but I know I can do better, so any tips are welcome!

EDIT: Also, if anyone has any suggestions for the extension, I'm open to those as well.

you are viewing a single comment's thread
view the rest of the comments
[–] [email protected] 3 points 3 weeks ago (1 children)

Hey, just a heads up - I updated the extension to add some rudimentary keyboard navigation. Your j and k are there, and also you can use m to toggle the child comments for the selected comment.

[–] [email protected] 4 points 3 weeks ago* (last edited 3 weeks ago) (2 children)

If you wanna tighten up your extension build process feel free to steal my build setup :) (https://github.com/Baizey/UniversalAutomaticCurrencyConverter )

It's using esbuild and I'm using it to bundle for both chrome and Firefox separately (they expect slightly different manifest files), has multi entry points support (background, content, options and popup) and builds for ts/js

[–] Lemminary 1 points 3 weeks ago* (last edited 3 weeks ago) (1 children)

bundle for both chrome and Firefox

You're a true warrior tolerating that. It gives me headaches.

E: And you even set up Storybook? Respect.

[–] [email protected] 2 points 3 weeks ago

Oh yeah, but the current manifest v3 difference between Firefox and Chrome is peanuts compared to when I had to have custom logic around regex as they didn't support the same set of regex specs. Fuck that

[–] [email protected] 1 points 3 weeks ago* (last edited 3 weeks ago)

Your repo made me want to convert the extension to TS, which I've never written in before.

So I did that, and I'm getting to learn a new syntax! Thanks :D