Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Croatian language support #4055

Merged
merged 3 commits into from
Jun 17, 2016

Conversation

thinkyhead
Copy link
Member

Cleanup and filling out of #4048

@jbrazio
Copy link
Contributor

jbrazio commented Jun 16, 2016

@thinkyhead I had already submitted this with #4052.

@thinkyhead
Copy link
Member Author

@jbrazio Sorry, I didn't notice. You might want to incorporate some of the changes I made here that differ from yours.

//#define SIMULATE_ROMFONT
#define DISPLAY_CHARSET_ISO10646_1 // use the better font on full graphic displays.

#ifndef WELCOME_MSG
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well...Is these much #ifndef needed?
I think that It's needed only in language_en.h...

Sample of alteration:
https://github.com/esenapaj/Marlin/commit/3b0c9199985db547536effe38b3a6f6132d21e2b

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aha, how did I miss that?

@thinkyhead thinkyhead merged commit 8d8cbef into MarlinFirmware:RCBugFix Jun 17, 2016
@thinkyhead thinkyhead deleted the rc_croatian_language branch June 17, 2016 00:22
thinkyhead added a commit that referenced this pull request Jun 17, 2016
Follow-up the PR #4055 (Croatian language support)
@thinkyhead thinkyhead mentioned this pull request Jul 8, 2016
@jbrazio jbrazio modified the milestone: 1.1.0 Jul 18, 2016
CONSULitAS pushed a commit to CONSULitAS/Marlin-K8200 that referenced this pull request Aug 18, 2016
・Update a forgotten Configuration.h of Cartesio
drewmoseley pushed a commit to drewmoseley/Marlin that referenced this pull request Nov 8, 2023
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.

3 participants