[REL] Commander Health Display v2.7.4 [61250]

Discussion in 'Mod Discussions' started by Raevn, August 14, 2013.

  1. Raevn

    Raevn Moderator Alumni

    Messages:
    4,226
    Likes Received:
    4,324
    Yep, my bad. I only set the position variables after a drag. It didn't occur to me that might not be sufficient :p
  2. tatsujb

    tatsujb Post Master General

    Messages:
    12,902
    Likes Received:
    5,385
    tried again. not only does it not show up but neither do the strat isons.
  3. Raevn

    Raevn Moderator Alumni

    Messages:
    4,226
    Likes Received:
    4,324
    Does the debugger show any errors? If you aren't sure how to check, with PA running, do the following
    1. Run PA\Coherent\Debugger\Win64\Debugger.exe (or Win32 as appropriate)
    2. Click "Go"
    3. Click the first item
    4. Look at the bottom right for a red/yellow icon, and click it if present
    Let me know any error messages that appear. Also, double-check ui_mods_list.js to ensure they've been enabled properly (if not, remove the file and let PAMM rebuild it).
  4. tatsujb

    tatsujb Post Master General

    Messages:
    12,902
    Likes Received:
    5,385
    yellow icon says 7
    this is the result.

    Attached Files:

    • dsf.txt
      File size:
      2.5 KB
      Views:
      9
  5. Raevn

    Raevn Moderator Alumni

    Messages:
    4,226
    Likes Received:
    4,324
    It doesn't look like you've enabled commander HP?
  6. proeleert

    proeleert Post Master General

    Messages:
    1,681
    Likes Received:
    1,656
    Let me know if you fixed this , I will enable it then for hotbuild preview :)
  7. Raevn

    Raevn Moderator Alumni

    Messages:
    4,226
    Likes Received:
    4,324
    So I've got a fix ready, but I've found a bug in PAMM to do with priority ordering of mods, and if I release the mod before fixing that it will cause all sorts of annoyance, so it wont be until tonight :(

    The bug with the Floating Framework is easily worked around though, as soon as you move the frame even once it won't occur again.
  8. proeleert

    proeleert Post Master General

    Messages:
    1,681
    Likes Received:
    1,656
    No hurries, code is ready in hotbuild2 just need to uncomment it.
    Will update tomorrow then , cause now I'm going to play with this cool build.
  9. tatsujb

    tatsujb Post Master General

    Messages:
    12,902
    Likes Received:
    5,385
    what? I did, I even checked the modlist files, rCommanderHP shows up everywhere it should.
  10. zaphodx

    zaphodx Post Master General

    Messages:
    2,350
    Likes Received:
    2,409
    I'm getting same problem as Tatsu I'm guessing we have a mod that is conflicting.

    Edit: Nope also done clean with no other mods apart from floating framework and settings manager
    Last edited: December 11, 2013
  11. tatsujb

    tatsujb Post Master General

    Messages:
    12,902
    Likes Received:
    5,385
    no look earlyer in the thread, I tried it with a freash repatch and mod folder completely emptied out (PAMM creates the modlist file)
  12. Raevn

    Raevn Moderator Alumni

    Messages:
    4,226
    Likes Received:
    4,324
    The log you attached doesn't show the rCommanderHP.js file being loaded at all. Can you double-check your ui_mod_list.js?

    Edit: Also check the UI tab in Settings, and let me know if you see the show/hide option for the Commander HP bar
  13. tatsujb

    tatsujb Post Master General

    Messages:
    12,902
    Likes Received:
    5,385
    i did both :( it's there.
  14. Raevn

    Raevn Moderator Alumni

    Messages:
    4,226
    Likes Received:
    4,324
    I can't replicate this issue :confused:
  15. DeathByDenim

    DeathByDenim Post Master General

    Messages:
    4,327
    Likes Received:
    2,125
    Actually it happens to me as well. I have both the floating frames and the settings mod installed, but the health display doesn't appear in either settings or during the game.
    I started up the debugger and when I go to the settings in PA it throws
    Code:
    Uncaught TypeError: Object #<SettingsViewModel> has no method 'addSetting_DropDown'
    Also it explicitly says that it loads rFloatFrame.js, rSettingsManager_global.js and rCommanderHP_settings.js
    There are no errors when in live_game though. It just doesn't show even though the debugger claims it loaded.
    (Oh, this is under Linux if that matters)
  16. Raevn

    Raevn Moderator Alumni

    Messages:
    4,226
    Likes Received:
    4,324
    The error message is because it's missing rSettingsManager.js, which should also be loaded on the settings screen - can you confirm it's definitely not loaded? What about in ui_mods_list.js?

    @tatsujb, @zaphodx & @DeathByDenim, Try these:

    Attached Files:

  17. DeathByDenim

    DeathByDenim Post Master General

    Messages:
    4,327
    Likes Received:
    2,125
    I tried the files on my office computer. The commander health now does show in live_game, but still not in settings. However, when I install the old mods from pamm again, it still works...? Maybe it's something specific to my home computer. I'll try that later. At home I do have a slightly lower resolution, 1600 x 900 instead of 1680 x 1050, but that shouldn't really matter, eh?

    I didn't see rSettingsManager.js in the Debugger at home though, but I'll check later, because I can't get the Debugger to work at work (seems to depend on an older version of PA). ui_mods_list.js was definitely loaded, because rModsList did work for example. Also rSettingsManager.js is present in ui_mods_list.js under settings:
    Code:
        'settings': [
            /* PAMM BEGIN (do not edit below) */
            /* Settings Manager */
            '../../mods/rSettingsManager/rSettingsManager.js',
            '../../mods/rSettingsManager/rSettingsManager.css',
            /* Commander Health Display */
            '../../mods/rCommanderHP/rCommanderHP_settings.js',
            /* Floating Framework */
            '../../mods/rFloatFrame/rFloatFrame_settings.js'
            /* PAMM END (do not edit above) */
        ],
    
  18. zaphodx

    zaphodx Post Master General

    Messages:
    2,350
    Likes Received:
    2,409
  19. Raevn

    Raevn Moderator Alumni

    Messages:
    4,226
    Likes Received:
    4,324
    Ah, that's good to hear. I found a whole bunch of edge case bugs. The difficulty working with anything to do with settings is as soon as it works once, it tends to keep working, even if you break the part that originally set the setting. This is why it would continue to work after rolling back to the old version.

    I also (embarassingly) failed to use my own mod correctly and didn't call initialSettingValue() to provide a default value for when to show the commander HP bar before using it :oops:. Not sure why rSettingsManager.js isn't working for you though.
  20. Raevn

    Raevn Moderator Alumni

    Messages:
    4,226
    Likes Received:
    4,324
    Version 2.7.1 Released
    • Bugfix - Initial value of setting "commander_hp_display_show" not set.
    • Bugfix - Default value for preferredCommander now included
    Please also update the Floating Framework mod.

Share This Page