Opened 15 years ago
Closed 15 years ago
#5850 closed enhancement (fixed)
Terminal localization patch
Reported by: | Karvjorm | Owned by: | stippi |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | Applications/Terminal | Version: | R1/alpha1 |
Keywords: | Terminal localization ISO 8859-15 | Cc: | Karvjorm |
Blocked By: | Blocking: | ||
Platform: | All |
Description
Hi,
I did not have mentioned that I have previously developed embedded systems programming applications (9 years) using Neuro C language. It is a subset of C with some communication objects. Also I have maintained an embedded C++ application running in Intel386 EX processors, if I recall it correct.
I have here a Terminal localization patch. PLS, can somebody delivery it to hta and make necessary modifications for implementation?
BTW, could somebody also add ISO 8859-15 support for Terminal: http://fi.wikipedia.org/wiki/ISO_8859-15 ? It replaced ISO 8859-1 when Euro symbol was needed.
Attachments (2)
Change History (4)
by , 15 years ago
Attachment: | TerminalLocalizationRevision36520.patch added |
---|
comment:1 by , 15 years ago
Owner: | changed from | to
---|---|
Status: | new → in-progress |
Patch looks mostly good. A few things you could fix:
src/apps/terminal/Shell.cpp:515/519, the part "alert --stop" should not be part of the translated string, otherwise it's too easy to break the functionality in translations. That is unless we want to translate command line options for apps like alert. ;-)
src/apps/terminal/TermView.cpp:1857, can you change the string "Secondary Mouse Button Drop Menu" to "Secondary mouse button drop menu"? We changed most strings in Haiku to sentence case a while back. This string won't usually be seen by users, since it's the internal name of the menu, but it's possible to see it as window title for example in ProcessController.
Lastly, I am missing an important detail. Somewhere you should have a BCatalog fAppCatalog variable declared, most probably in the Terminal BApplication class and load it at init time. Without this, Terminal should crash.
Thanks a lot for the work, I will be happy to submit it with the above changes! :-)
by , 15 years ago
Attachment: | TerminalLocalization2.patch added |
---|
A fixed Terminal localization patch
comment:2 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | in-progress → closed |
Localization patch for the Terminal application