• 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

            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”.

          • 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.