ddm999 closed issue fortressforever/fortressforever#92 ('With require() replacing IncludeScript(), included files are not logged.')

View on Github

ddm999 commented on issue fortressforever/fortressforever#92 ('With require() replacing IncludeScript(), included files are not logged.'): 'Fair enough.. (Also I had no idea those were actually posted because the script behind GitHub...'

View on Github

FDA- commented on issue fortressforever/fortressforever#93 ('Make weapon slots consistent (3 weapon system revisited)'): 'I agree with Aftershock about demo on all points. I also would like to see the spy keep his nail...'

View on Github

ddm999 commented on issue fortressforever/fortressforever#93 ('Make weapon slots consistent (3 weapon system revisited)'): 'I agree with FDA on this. Making it have 3 weapon slots for all classes just hides away parts from...'

View on Github

ddm999 opened issue fortressforever/fortressforever#94 ('Modify the map guide options on team select.')

View on Github

squeek502 commented on issue fortressforever/fortressforever#93 ('Make weapon slots consistent (3 weapon system revisited)'): '@FDA-:. > The idea sounds very clunky, and could confuse players into thinking "hold click to...'

View on Github

squeek502 commented on issue fortressforever/fortressforever#94 ('Modify the map guide options on team select.'): 'Makes sense.'

View on Github

FDA- commented on issue fortressforever/fortressforever#93 ('Make weapon slots consistent (3 weapon system revisited)'): '" Should he stand back and kill an SG from far away or should he use his unique abilities to get...'

View on Github

AfterShockFF commented on issue fortressforever/fortressforever#90 ('Default hud_fastswitch to 1 and add hud_weaponselect cvar'): 'Sounds like a great compromise. :+1: '

View on Github

AfterShockFF commented on the diff of pull request fortressforever/fortressforever#89@e9c4db ('Fixed CFFCrosshairOptions::UpdateSliders assert and did some cleanup'): 'this iCurrentWeapon is a bit confusing now - is this actually currentCrosshair ?'

View on Github

AfterShockFF commented on issue fortressforever/fortressforever#89 ('Fixed CFFCrosshairOptions::UpdateSliders assert and did some cleanup'): 'Minor suggestion otherwise all good :+1: '

View on Github

AfterShockFF commented on the diff of pull request fortressforever/fortressforever#88@ea856b ('Generate <classname>.cfg/userconfig.cfg files at runtime if they don't exist'): 'whats MOD here?'

View on Github

AfterShockFF commented on the diff of pull request fortressforever/fortressforever#88@ea856b ('Generate <classname>.cfg/userconfig.cfg files at runtime if they don't exist'): 'whats "rb"?'

View on Github

AfterShockFF commented on the diff of pull request fortressforever/fortressforever#88@ea856b ('Generate <classname>.cfg/userconfig.cfg files at runtime if they don't exist'): 'This whole loop could do with a bit more readability help. Hungarian prefixes hurt when it gets to...'

View on Github

AfterShockFF commented on the diff of pull request fortressforever/fortressforever#88@ea856b ('Generate <classname>.cfg/userconfig.cfg files at runtime if they don't exist'): 'I think you can invert this "if" statement and just return early if the file doesnt exist, reduces...'

View on Github

AfterShockFF commented on the diff of pull request fortressforever/fortressforever#88@ea856b ('Generate <classname>.cfg/userconfig.cfg files at runtime if they don't exist'): 'Again you could return early here and invert the if statement, reduces nesting'

View on Github

AfterShockFF commented on issue fortressforever/fortressforever#88 ('Generate <classname>.cfg/userconfig.cfg files at runtime if they don't exist'): 'This is generally a good idea. The code could do with a slight cleanup but otherwise all good :+1: '

View on Github

AfterShockFF commented on issue fortressforever/fortressforever#85 ('Scout movement weapon'): 'The only other thing we werent quite happy with, design mechanics wise, (if I remember correctly,...'

View on Github

AfterShockFF commented on the diff of pull request fortressforever/fortressforever#85@c9bfd4 ('Scout movement weapon'): 'i suspect we don't need both?'

View on Github

AfterShockFF commented on the diff of pull request fortressforever/fortressforever#85@c9bfd4 ('Scout movement weapon'): 'We should probably try to get rid of these useless comments on new files, at least we should try to...'

View on Github