Today's palette is r-place-2023 "r/place 2023 Palette"
Upon trying to save it as a preset, this throws an error in line 145 of `lospec-palette-importer.lua` "Error writing to palette file!"
I suppose this happens because of the forward slash in the palette name. I'm on Linux if this information is relevant.
There are otherwise zero issues on Linux I've encountered so far.
Is the palette name in general sanitized? It looks like there are some other crazy symbols that are also disallowed on Windows, e.g. the pipe, like this palette: https://lospec.com/palette-list/fourlines-22
Glad to hear it! Some Linux users have reported that the extension trying to use curl is broken for them - I haven’t nailed down a good fix, and my Linux setup is just a separate partition on this MacBook, so it’s maybe not 100%.
The palette name (as provided by the user) is “sluggified” using a lua port of Skeddles’ own “sluggify” that he uses for Lospec proper, but (as far as I remember) the raw name is used for saved palette files - I’ll look into sanitization for those too.
FWIW, I tried a different r/place palette and encountered the same issue, so I think you’re right on about the slash being the issue:
And I didn’t have any issue saving “|||| 22 PALETTE” on MacOS, so I’ll chalk that up to platform-specific shenanigans.
What I might do is fall back to saving the palette using its slug if using the actual name fails (and if that fails, something else is up).
Thanks for the report!
