• Edie [it/its]
      link
      fedilink
      arrow-up
      5
      arrow-down
      1
      ·
      2 years ago

      Oh fucking lord. I made the mistake of attempting to look into the characters, so I could make a PR. It’s unicode, it’s fucking unicode, of course it’s a nightmare.

      • 🏳️‍⚧️ Elara ☭OPM
        link
        fedilink
        arrow-up
        6
        ·
        2 years ago

        Yeah lol. It’s not a very difficult fix really. Rather than checking if the string contains any invisible characters, it should check if it only contains invisible characters. The way they’re doing it now unintentionally breaks a lot of stuff (breaking stuff unintentionally is a pretty common thing with unicode lol).

        • Edie [it/its]
          link
          fedilink
          arrow-up
          3
          arrow-down
          1
          ·
          2 years ago

          It’s honestly easier to just check if it starts with those.

        • Edie [it/its]
          link
          fedilink
          arrow-up
          3
          arrow-down
          1
          ·
          2 years ago

          Hey, at least I get to laugh at unicode: “combining grapheme joiner”, “the name of this character is misleading; it does not actually join graphemes”.