Skip to content

Conversation

Stichoza
Copy link

According to lab.flipper.net, files are not allowed to contain commas. It displays an error right when you click rename on files. To avoid this error, it would be good to replace comma with dashes.

Example

image image

What's new

  • Replaced commas with dashes

For the reviewer

  • I've uploaded the firmware with this patch to a device and verified its functionality
  • I've confirmed the bug to be fixed / feature to be stable

According to lab.flipper.net, files are not allowed to contain commas. It displays an error right when you click rename on files. To avoid this error, it would be good to replace comma with dashes.
@WillyJL
Copy link
Member

WillyJL commented Jul 29, 2025

What a goofy limitation, can't think of any reason why commas would be an issue, maybe they just forgot to add it

@Stichoza
Copy link
Author

Yeah, but dashes are more common in datetime strings than commas anyways so idk

@WillyJL
Copy link
Member

WillyJL commented Jul 30, 2025

They are for dates, I would've used colons but that's an illegal character. Didn't use dots as they are for the extension. Commas are thin so the text doesn't go off screen too much and also it differentiates from the date and extension

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants