|
|
Wow, would you believe I've never tried that control before? (or Turn Off Auto-Targeting of Subsystems). Useful! And it's in retail even! (Just so everyone ELSE knows, this has nothing to do with my build, it's just more desirable to use this control in my build ^^ ) How about the names: "Deactivate Targeting" "Deactivate Subsystem Targeting" Other suggestions? It's only the names we're changing, to make it more clear what the controls do (switch target to nothing). |
|
|
I think this could be a cool tweak for 3.6.9. Backslash's names are good but I prefer "Turn off" rather than "Deactivate" edited on: 06-11-06 13:02 |
|
|
*bump* Just to remember SCP to tweak this serious and evil bug before 3.6.9. launch. Although I doubt you will be able to get rid of it with less than 3 years of hard team work. Just a joke, don't get me wrong :o |). edited on: 07-09-06 14:11 |
|
|
Fixed. ;) |
|
|
In 3.6.9. RC6 I still see: Alt - T: Turn off auto-targeting Alt - S: Turn off auto-targeting of subsystems |
|
|
Are you sure you're using RC6? Because it's definitely fixed; I just checked. |
|
2006-08-08 15:02
|
Binding Screen.jpg (38,555 bytes) |
|
|
I have just re-extracted fs2_open_3_6_9.exe from 3.6.9-RC6.rar, I have created even a new pilot and I still get the old names. I attach a screenshot. |
|
|
Ugh. It's using the localized version in strings.tbl. I guess we have two alternatives. Either have FSO use the version in the code when doing English translations, or fix strings.tbl in the mediaVPs. |
|
|
I think the best fix you can do is: + Add support for a modular string tables. + Add one "Targeting-str.tbm" file to the new media vp with the fixed text in English, German and French (any translators please?). (If something isn't hardcoded, I think it's a bad idea to hardcode it.) But maybe it's too much work to solve such little issue. In this case just put a fixed Strings.tbl in next Media vp. (I didn't know about Strings.tbl so I'm fixing my vp just now) |
|
|
Adding support for modular strings.tbl shouldn't be a big deal, I can do that without much trouble. It's only part of the translation system though. The problem is that tstrings.tbl can't really be made modular without a rather large rewrite of the localization code. That tbl isn't actually parsed, it's opened and accessed at lookup time by seeking in the file. This reason for that is simple, it's a very large tbl with an arbitrary number of entries. Making it modular wouldn't be an easy task if it was done even remotely right. If everyone would be happy with one tbl and not the other then I'll work on making strings.tbl modular later today. |
|
|
I seriously don't think it's necessary. How often is strings.tbl used (by third-party campaigns that is), anyway? The only reason for making it modular would be to fix this bug, and that's overkill IMO. I doubt this is even worth a mediaVP fix, considering that new keys may be added or changed in the future. I strongly recommend to just add something to XSTR that uses the version in the code if the language is English, then close this bug and that'll be the end of the matter. :) |
|
|
The benefit of modular strings.tbl is that we can localize our own strings, not just the original :V: ones. Making it modular would allow us to not only replace but also add translated strings. Everything that we have added to the code needs to be translated too, but we would have to include a strings.tbl in the MediaVPs for that, or hard code the translations (which is a very bad idea). A modular strings.tbl wouldn't be just for this bug. And strings.tbl is only for internal code translations, so it wouldn't really be used by mods anyway, they would just make use of tstrings.tbl if anything. The XSTR hack, unless it's a flag set per call, would be a bad idea. Not all of the strings in the code are exactly what is in the tbl, so we would actually have to go check everything to make sure that the strings in the code are correct. Plus, the strings in the code are mainly for a last ditch fallback when the line doesn't exist in the tbl, and as a simple coder reference for what should be said. Using the in-code strings for anything should actually be considered a bug, and not as a fix for anything. |
|
|
*grumble grumble* Okay, well at the very least we should have a built-in modular table, in def_files, to correct those key descriptions I fixed in CVS. Hopefully, this is a compromise and doesn't fall too far afoul of hard coding. :) |
|
|
Is this actually such a big thing that it needs to be hard coded? It doesn't seem that way to me, but, whatever. Hard coding translations, not to mention translation fixes, is never a good idea, and the code is already bad enough about that crap. Adding it to def_files means that a coder will have to take the time to do the non-English translations, or at least make the changes submitted by others. A significant portion of the FS2 user base is non-English so only having the correct English translation still leaves the bug, and this wouldn't have actually been "fixed" at all. I guess I'll try and take care of all of that when I make strings.tbl modular though, just so no one else will have to deal with it later. And so I don't have to deal with the non-English guys going on a rampage again, since I can't always understand what the hell they are talking about. edited on: 08-09-06 17:34 |
|
|
IMHO it's such a *little* thing that it needs to be hard-coded. I made about a dozen changes total, and the only one where I actually changed the phrasing was the "Turn-off Auto-Targeting" command mentioned here. All the other changes were merely changes in capitalization to make the whole thing consistent. That hardly needs to be duplicated for the other locales, IMHO; they're just nitpicks. |
|
2006-08-15 18:41
|
|
|
|
IMO, do not hard-code anything. I've just upload a fixed Strings.tbl at least for English (French and German are unchanged). Add it to next vp distribution and forget about this little bug, (it doesn't worth your effort). |
|
|
You're not the Grammar Inquisition. :p I fixed several typos in addition to the one in this bug. |
|
|
I'm not native English speaker so I'm not risking checking any other's grammar. ;) OTOH, I made TWO changes, not only one: 517 and 521 strings. |
|
|
Okay, taylor, are there any plans to do anything with this, or shall we mark it as not fixable? |
|
|
Reminder sent to: taylor Just a question. ;) |
|
|
Yeah, sorry, forgot to ever get back to this. I'm in the process of making the strings stuff modular so that stuff like this can be easily fixed. That's a lot easier said that done though, and I just haven't gotten it working yet. It's sort of working for strings.tbl, but since tstrings.tbl works differently it's quite a bit more difficult to manage that one. I might just finish getting strings.tbl working (since that would address this bug) and deal with tstrings.tbl at a later date, if there is a real need for it. |
|
|
Modular strings.tbl support is now in CVS (*-lcl.tbm). Consider this fixed? |
|
|
Cool. :) Not "fixed" fixed, unfortunately, as we still need new strings. But the good news is that it's no longer our concern. :) I'll reassign it to DaBrain. |
|
2007-04-12 06:28
|
|
|
|
Probably should have done this before, but just attached a tbm to make use of the code fix. Should be something that DaBrain can go ahead and add to the MediaVPs though, since it won't hurt anything for builds that don't support it yet. |
|
|
Good, but the mediaVPs should include a few more changes IMHO. Here are the my commits from earlier in this thread: http://fs2source.warpcore.org/cgi-bin/cvsweb/cvsweb.cgi/fs2_open/code/controlconfig/controlsconfigcommon.cpp.diff?r1=text&tr1=2.14&r2=text&tr2=2.15 http://fs2source.warpcore.org/cgi-bin/cvsweb/cvsweb.cgi/fs2_open/code/controlconfig/controlsconfigcommon.cpp.diff?r1=text&tr1=2.16&r2=text&tr2=2.17 |
|
|
"mv_strings-lcl.tbm" added to MV_Core. Can be closed after the MediaVP 3.6.10 release. |
|
|
DaBrain, did you add the extra fixes in the links that I mentioned? |
|
|
Not yet. I'm not really sure how to 'translate' that into the tables. But I didn't take a really close look at it. |
|
|
I posted DaBrain the fix for this one which includes the files Goober mentioned later. Should be in the 3.6.10 Mediavps (not in beta) |
|
|
Fixed in 3.6.10 mediavps (not in beta) |
- Anonymous
