vankuzu Posted May 31, 2014 Share Posted May 31, 2014 Так никто специально и не оформлял. Есть предложения?В ночнушке под 0.9.1 все нормально сделано, так и оставьте. Link to comment Short link Share on other sites More sharing options...
seriych Posted May 31, 2014 Share Posted May 31, 2014 Коммит увидел, все ок.я только во вшитом конфиге не стал менять, не уверен, что правильно понял, как там с кругами правильно прописывать, а компилятора не стоит Link to comment Short link Share on other sites More sharing options...
sirmax Posted May 31, 2014 Share Posted May 31, 2014 (edited) я только во вшитом конфиге не стал менять, не уверен, что правильно понял, как там с кругами правильно прописывать, а компилятора не стоит А там и не надо. Вернее круг 445м я добавлю, а остальное не нужно. Edited May 31, 2014 by sirmax Link to comment Short link Share on other sites More sharing options...
seriych Posted May 31, 2014 Share Posted May 31, 2014 @sirmax, На макросы с нормализацией типа {{hp:100}} не оказывает влияния параметр "allowHpInPanelsAndMinimap": false, они работают даже при false Link to comment Short link Share on other sites More sharing options...
sirmax Posted May 31, 2014 Share Posted May 31, 2014 @sirmax, На макросы с нормализацией типа {{hp:100}} не оказывает влияния параметр "allowHpInPanelsAndMinimap": false, они работают даже при false И не должно. Этот параметр только включает/выключает передачу изменений HP в песочницу с ушами и миникартой. Link to comment Short link Share on other sites More sharing options...
seriych Posted May 31, 2014 Share Posted May 31, 2014 И не должно. Этот параметр только включает/выключает передачу изменений HP в песочницу с ушами и миникартой.при "allowHpInPanelsAndMinimap": false значение макроса {{hp}} пусто, а {{hp:100}} - число Link to comment Short link Share on other sites More sharing options...
sirmax Posted May 31, 2014 Share Posted May 31, 2014 при "allowHpInPanelsAndMinimap": false значение макроса {{hp}} пусто, а {{hp:100}} - число Так и при "allowHpInPanelsAndMinimap": true так же. Link to comment Short link Share on other sites More sharing options...
seriych Posted May 31, 2014 Share Posted May 31, 2014 Так и при "allowHpInPanelsAndMinimap": true так же.при true оба макроса выдают числа, как и должно быть Link to comment Short link Share on other sites More sharing options...
sirmax Posted May 31, 2014 Share Posted May 31, 2014 при true оба макроса выдают числа, как и должно быть Для врагов тоже? Link to comment Short link Share on other sites More sharing options...
seriych Posted May 31, 2014 Share Posted May 31, 2014 Для врагов тоже?Да. вот true: вот false: было бы логичным иметь одинаковое поведение обоих макросов: - при false пустое значение для {{hp:100}} - при true числовое значение для {{hp}} еще не светившихся танков Link to comment Short link Share on other sites More sharing options...
sirmax Posted June 1, 2014 Share Posted June 1, 2014 было бы логичным иметь одинаковое поведение обоих макросов: - при false пустое значение для {{hp:100}} - при true числовое значение для {{hp}} еще не светившихся танков Так в твоем примере как раз для не светившихся пусто в обоих случаях. При выключенном параметре можно считать, что все не светились. Там только 2 состояния возможны - живой/мертвый. Link to comment Short link Share on other sites More sharing options...
seriych Posted June 1, 2014 Share Posted June 1, 2014 Так в твоем примере как раз для не светившихся пусто в обоих случаях. А должно быть непусто для несветившихся при true. При выключенном параметре можно считать, что все не светились. Дак зачем это надо? Это только мешает, ибо например не позволяет заюзать один параметр "allowHpInPanelsAndMinimap" как переключатель отображения hp. Link to comment Short link Share on other sites More sharing options...
sirmax Posted June 1, 2014 Share Posted June 1, 2014 А должно быть непусто для несветившихся при true. Так а что туда тогда писать? Пока не светился, не известно сколько у него ХП. Дак зачем это надо? Это только мешает, ибо например не позволяет заюзать один параметр "allowHpInPanelsAndMinimap" как переключатель отображения hp. Такой цели как бы и не стояло. Эта опция была введена только из-за того, что обновление ХП тормозит. Link to comment Short link Share on other sites More sharing options...
seriych Posted June 1, 2014 Share Posted June 1, 2014 Так а что туда тогда писать? Пока не светился, не известно сколько у него ХП. Тоже, что пишется для {{hp:100}}, только без нормализации. Такой цели как бы и не стояло. Эта опция была введена только из-за того, что обновление ХП тормозит. Это понятно, но зачем нужно обратное? А так хоть польза будет очевидная Link to comment Short link Share on other sites More sharing options...
sirmax Posted June 1, 2014 Share Posted June 1, 2014 Тоже, что пишется для {{hp:100}}, только без нормализации. С чего бы? Народ сейчас пишет {{hp|----}}, и это лучше, чем выдавать значение, которое может быть неправильным. В случае с нормализацией это допустимо, так как там цель другая - нарисовать полоску определенной ширины. Link to comment Short link Share on other sites More sharing options...
seriych Posted June 1, 2014 Share Posted June 1, 2014 С чего бы? Народ сейчас пишет {{hp|----}}, и это лучше, чем выдавать значение, которое может быть неправильным. В случае с нормализацией это допустимо, так как там цель другая - нарисовать полоску определенной ширины. ок, позиция понятна. Что на счет сделать: - при false пустое значение для {{hp:100}} и т.п. Link to comment Short link Share on other sites More sharing options...
sirmax Posted June 1, 2014 Share Posted June 1, 2014 А зачем вообще использовать эти макросы если они не работают? Link to comment Short link Share on other sites More sharing options...
seriych Posted June 1, 2014 Share Posted June 1, 2014 А зачем вообще использовать эти макросы если они не работают? заюзать один параметр "allowHpInPanelsAndMinimap" как переключатель отображения hp. Link to comment Short link Share on other sites More sharing options...
sirmax Posted June 1, 2014 Share Posted June 1, 2014 А зачем вообще использовать эти макросы если они не работают? заюзать один параметр "allowHpInPanelsAndMinimap" как переключатель отображения hp. Ладно, убедил. Хуже точно не будет. :) 1 @ Link to comment Short link Share on other sites More sharing options...
TornadoCat Posted June 1, 2014 Share Posted June 1, 2014 Сир, а переименование extraTextFields в extraFields было где то указано ? Link to comment Short link Share on other sites More sharing options...
Recommended Posts