vankuzu 28 #187948 Posted May 31, 2014 Так никто специально и не оформлял. Есть предложения?В ночнушке под 0.9.1 все нормально сделано, так и оставьте. Share this post Link to post Short link Share on other sites
seriych 2,178 #187955 Posted May 31, 2014 Коммит увидел, все ок.я только во вшитом конфиге не стал менять, не уверен, что правильно понял, как там с кругами правильно прописывать, а компилятора не стоит Share this post Link to post Short link Share on other sites
sirmax 5,499 #187959 Posted May 31, 2014 (edited) я только во вшитом конфиге не стал менять, не уверен, что правильно понял, как там с кругами правильно прописывать, а компилятора не стоит А там и не надо. Вернее круг 445м я добавлю, а остальное не нужно. Edited May 31, 2014 by sirmax Share this post Link to post Short link Share on other sites
seriych 2,178 #187980 Posted May 31, 2014 @sirmax, На макросы с нормализацией типа {{hp:100}} не оказывает влияния параметр "allowHpInPanelsAndMinimap": false, они работают даже при false Share this post Link to post Short link Share on other sites
sirmax 5,499 #187983 Posted May 31, 2014 @sirmax, На макросы с нормализацией типа {{hp:100}} не оказывает влияния параметр "allowHpInPanelsAndMinimap": false, они работают даже при false И не должно. Этот параметр только включает/выключает передачу изменений HP в песочницу с ушами и миникартой. Share this post Link to post Short link Share on other sites
seriych 2,178 #187986 Posted May 31, 2014 И не должно. Этот параметр только включает/выключает передачу изменений HP в песочницу с ушами и миникартой.при "allowHpInPanelsAndMinimap": false значение макроса {{hp}} пусто, а {{hp:100}} - число Share this post Link to post Short link Share on other sites
sirmax 5,499 #187989 Posted May 31, 2014 при "allowHpInPanelsAndMinimap": false значение макроса {{hp}} пусто, а {{hp:100}} - число Так и при "allowHpInPanelsAndMinimap": true так же. Share this post Link to post Short link Share on other sites
seriych 2,178 #187993 Posted May 31, 2014 Так и при "allowHpInPanelsAndMinimap": true так же.при true оба макроса выдают числа, как и должно быть Share this post Link to post Short link Share on other sites
sirmax 5,499 #187994 Posted May 31, 2014 при true оба макроса выдают числа, как и должно быть Для врагов тоже? Share this post Link to post Short link Share on other sites
seriych 2,178 #187997 Posted May 31, 2014 Для врагов тоже?Да. вот true: вот false: было бы логичным иметь одинаковое поведение обоих макросов: - при false пустое значение для {{hp:100}} - при true числовое значение для {{hp}} еще не светившихся танков Share this post Link to post Short link Share on other sites
sirmax 5,499 #188019 Posted June 1, 2014 было бы логичным иметь одинаковое поведение обоих макросов: - при false пустое значение для {{hp:100}} - при true числовое значение для {{hp}} еще не светившихся танков Так в твоем примере как раз для не светившихся пусто в обоих случаях. При выключенном параметре можно считать, что все не светились. Там только 2 состояния возможны - живой/мертвый. Share this post Link to post Short link Share on other sites
seriych 2,178 #188029 Posted June 1, 2014 Так в твоем примере как раз для не светившихся пусто в обоих случаях. А должно быть непусто для несветившихся при true. При выключенном параметре можно считать, что все не светились. Дак зачем это надо? Это только мешает, ибо например не позволяет заюзать один параметр "allowHpInPanelsAndMinimap" как переключатель отображения hp. Share this post Link to post Short link Share on other sites
sirmax 5,499 #188034 Posted June 1, 2014 А должно быть непусто для несветившихся при true. Так а что туда тогда писать? Пока не светился, не известно сколько у него ХП. Дак зачем это надо? Это только мешает, ибо например не позволяет заюзать один параметр "allowHpInPanelsAndMinimap" как переключатель отображения hp. Такой цели как бы и не стояло. Эта опция была введена только из-за того, что обновление ХП тормозит. Share this post Link to post Short link Share on other sites
seriych 2,178 #188036 Posted June 1, 2014 Так а что туда тогда писать? Пока не светился, не известно сколько у него ХП. Тоже, что пишется для {{hp:100}}, только без нормализации. Такой цели как бы и не стояло. Эта опция была введена только из-за того, что обновление ХП тормозит. Это понятно, но зачем нужно обратное? А так хоть польза будет очевидная Share this post Link to post Short link Share on other sites
sirmax 5,499 #188040 Posted June 1, 2014 Тоже, что пишется для {{hp:100}}, только без нормализации. С чего бы? Народ сейчас пишет {{hp|----}}, и это лучше, чем выдавать значение, которое может быть неправильным. В случае с нормализацией это допустимо, так как там цель другая - нарисовать полоску определенной ширины. Share this post Link to post Short link Share on other sites
seriych 2,178 #188044 Posted June 1, 2014 С чего бы? Народ сейчас пишет {{hp|----}}, и это лучше, чем выдавать значение, которое может быть неправильным. В случае с нормализацией это допустимо, так как там цель другая - нарисовать полоску определенной ширины. ок, позиция понятна. Что на счет сделать: - при false пустое значение для {{hp:100}} и т.п. Share this post Link to post Short link Share on other sites
sirmax 5,499 #188051 Posted June 1, 2014 А зачем вообще использовать эти макросы если они не работают? Share this post Link to post Short link Share on other sites
seriych 2,178 #188052 Posted June 1, 2014 А зачем вообще использовать эти макросы если они не работают? заюзать один параметр "allowHpInPanelsAndMinimap" как переключатель отображения hp. Share this post Link to post Short link Share on other sites
sirmax 5,499 #188186 Posted June 1, 2014 А зачем вообще использовать эти макросы если они не работают? заюзать один параметр "allowHpInPanelsAndMinimap" как переключатель отображения hp. Ладно, убедил. Хуже точно не будет. :) 1 Share this post Link to post Short link Share on other sites
TornadoCat 1,027 #188197 Posted June 1, 2014 Сир, а переименование extraTextFields в extraFields было где то указано ? Share this post Link to post Short link Share on other sites