Skip to content

Disable UTF_FILENAME_SUPPORT by default#20135

Merged
thinkyhead merged 5 commits intoMarlinFirmware:bugfix-2.0.xfrom
thisiskeithb:pr/disable_UTF_FILENAME_SUPPORT_default
Nov 14, 2020
Merged

Disable UTF_FILENAME_SUPPORT by default#20135
thinkyhead merged 5 commits intoMarlinFirmware:bugfix-2.0.xfrom
thisiskeithb:pr/disable_UTF_FILENAME_SUPPORT_default

Conversation

@thisiskeithb
Copy link
Member

@thisiskeithb thisiskeithb commented Nov 14, 2020

Description

With UTF_FILENAME_SUPPORT enabled by default, it causes issues with LCD controllers that don't support UTF-8 (Non-character controllers like REPRAP_DISCOUNT_FULL_GRAPHIC_SMART_CONTROLLER, CR10_STOCKDISPLAY, etc.)

I noticed the issue when a hidden macOS folder was partially displayed on the LCD & not my "printed" folder which I toss printed g-code into once they're done:

With UTF_FILENAME_SUPPORT enabled With UTF_FILENAME_SUPPORT disabled
UTF_FILENAME_SUPPORT enabled UTF_FILENAME_SUPPORT disabled

Benefits

Opt-in UTF_FILENAME_SUPPORT vs. opt-out for configs.

Related Issues

Issue #20114 and PRs #20087, #20121, #20115

@thisiskeithb
Copy link
Member Author

Would a sanity check be preferred here instead?

@thinkyhead thinkyhead merged commit f9e5433 into MarlinFirmware:bugfix-2.0.x Nov 14, 2020
@thisiskeithb thisiskeithb deleted the pr/disable_UTF_FILENAME_SUPPORT_default branch November 14, 2020 02:11
FhlostonParadise pushed a commit to FhlostonParadise/Marlin that referenced this pull request Nov 21, 2020
Kannix2005 pushed a commit to Kannix2005/Marlin-1 that referenced this pull request Dec 7, 2020
vgadreau pushed a commit to vgadreau/Marlin that referenced this pull request Dec 9, 2020
tharts pushed a commit to tharts/Marlin that referenced this pull request Jan 6, 2021
kpishere pushed a commit to kpishere/Marlin that referenced this pull request Feb 19, 2021
W4tel-BiDi pushed a commit to W4tel-BiDi/Marlin that referenced this pull request Apr 5, 2021
thinkyhead pushed a commit that referenced this pull request Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants