-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dualwield спрайты #253
Dualwield спрайты #253
Changes from 1 commit
1f63cb4
d7e76ad
2c00310
21dd7d8
e07255b
20cd963
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -24,6 +24,14 @@ | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
"name": "equipped-BACKPACK", | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
"directions": 4 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
"name": "wielded-inhand-left", | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
"directions": 4 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
"name": "wielded-inhand-right", | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
"directions": 4 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Comment on lines
+27
to
+34
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Изменения одобрены, но есть предложение по улучшению. Новые состояния "wielded-inhand-left" и "wielded-inhand-right" корректно добавлены и соответствуют целям PR по добавлению спрайтов для двойного владения оружием. Свойство "directions" со значением 4 согласуется с другими состояниями в файле. Для улучшения читаемости и организации кода предлагаю сгруппировать связанные состояния вместе. Например: {
"name": "inhand-left",
"directions": 4
},
{
"name": "inhand-right",
"directions": 4
},
+{
+ "name": "wielded-inhand-left",
+ "directions": 4
+},
+{
+ "name": "wielded-inhand-right",
+ "directions": 4
+},
{
"name": "equipped-BACKPACK",
"directions": 4
-},
-{
- "name": "wielded-inhand-left",
- "directions": 4
-},
-{
- "name": "wielded-inhand-right",
- "directions": 4
} Это позволит легче находить и сравнивать связанные состояния в будущем. 📝 Committable suggestion
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
] | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -21,6 +21,14 @@ | |
{ | ||
"name": "equipped-BACKPACK", | ||
"directions": 4 | ||
}, | ||
{ | ||
"name": "wielded-inhand-left", | ||
"directions": 4 | ||
}, | ||
{ | ||
"name": "wielded-inhand-right", | ||
"directions": 4 | ||
Comment on lines
+24
to
+31
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Одобрено: Добавлены новые состояния для двойного владения оружием. Новые состояния "wielded-inhand-left" и "wielded-inhand-right" корректно добавлены в конфигурацию. Они соответствуют существующему стилю именования и имеют правильное количество направлений (4), что согласуется с другими состояниями в руках. Эти изменения хорошо поддерживают визуальное представление двойного владения оружием в игре, что соответствует целям PR. Для улучшения читаемости и организации кода, рекомендуется сгруппировать похожие состояния вместе. Например, можно расположить новые состояния сразу после соответствующих им обычных состояний "inhand-left" и "inhand-right". |
||
} | ||
] | ||
} |
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -34,6 +34,14 @@ | |||||||||||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||||||||||||
"name": "equipped-SUITSTORAGE", | ||||||||||||||||||||||||||||||||||||||||||||
"directions": 4 | ||||||||||||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||||||||||||
"name": "wielded-inhand-left", | ||||||||||||||||||||||||||||||||||||||||||||
"directions": 4 | ||||||||||||||||||||||||||||||||||||||||||||
}, | ||||||||||||||||||||||||||||||||||||||||||||
{ | ||||||||||||||||||||||||||||||||||||||||||||
"name": "wielded-inhand-right", | ||||||||||||||||||||||||||||||||||||||||||||
"directions": 4 | ||||||||||||||||||||||||||||||||||||||||||||
Comment on lines
+37
to
+44
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Изменения соответствуют целям PR и выглядят корректными. Добавление новых состояний "wielded-inhand-left" и "wielded-inhand-right" согласуется с целью PR по внедрению спрайтов для двойного владения оружием. Атрибут "directions" со значением 4 соответствует существующим состояниям "inhand-left" и "inhand-right". Для улучшения читаемости кода рекомендуется добавить пустую строку перед новыми состояниями: "name": "equipped-SUITSTORAGE",
"directions": 4
},
+
{
"name": "wielded-inhand-left",
"directions": 4
}, 📝 Committable suggestion
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||
} | ||||||||||||||||||||||||||||||||||||||||||||
] | ||||||||||||||||||||||||||||||||||||||||||||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,7 @@ | ||
{ | ||
"version": 1, | ||
"license": "CC-BY-SA-3.0", | ||
"copyright": "Taken/modified from tgstation at https://github.com/tgstation/tgstation/pull/41393/commits/1e56473177d0994d163c9edca3d13d6e5b640cc4, https://github.com/tgstation/tgstation/tree/master/icons/obj/weapons/guns backpack sprite by Peptide (copy of carbine), backpack sling sprite edited by Boaz1111", | ||
"copyright": "Taken/modified from tgstation at https://github.com/tgstation/tgstation/pull/41393/commits/1e56473177d0994d163c9edca3d13d6e5b640cc4, https://github.com/tgstation/tgstation/tree/master/icons/obj/weapons/guns backpack sprite by Peptide (copy of carbine), backpack sling sprite edited by Boaz1111, wield sprites by RiceMar1244", | ||
"size": { | ||
"x": 32, | ||
"y": 32 | ||
|
@@ -27,11 +27,19 @@ | |
"name": "inhand-right", | ||
"directions": 4 | ||
}, | ||
{ | ||
"name": "wielded-inhand-left", | ||
"directions": 4 | ||
}, | ||
{ | ||
"name": "wielded-inhand-right", | ||
"directions": 4 | ||
}, | ||
Comment on lines
+30
to
+37
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Новые состояния для спрайтов двуручного оружия одобрены, но рекомендуется улучшить организацию. Добавление состояний "wielded-inhand-left" и "wielded-inhand-right" соответствует целям PR и расширяет функциональность спрайтов оружия. Количество направлений (4) согласуется с существующими состояниями. Для улучшения читаемости кода рекомендуется переместить новые состояния ближе к соответствующим однорукимм состояниям. Предлагаемая структура: {
"name": "inhand-left",
"directions": 4
},
{
"name": "wielded-inhand-left",
"directions": 4
},
{
"name": "inhand-right",
"directions": 4
},
{
"name": "wielded-inhand-right",
"directions": 4
}, |
||
{ | ||
"name": "equipped-BACKPACK", | ||
"directions": 4 | ||
}, | ||
{ | ||
{ | ||
"name": "equipped-SUITSTORAGE", | ||
"directions": 4 | ||
} | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,7 @@ | ||
{ | ||
"version": 1, | ||
"license": "CC-BY-SA-3.0", | ||
"copyright": "Taken from tgstation at https://github.com/tgstation/tgstation/commit/1b6831dab1e1a74c0d91f2229adb87abbb089d31, backpack sprite by Peptide, backpack sling sprite edited by Boaz1111", | ||
"copyright": "Taken from tgstation at https://github.com/tgstation/tgstation/commit/1b6831dab1e1a74c0d91f2229adb87abbb089d31, backpack sprite by Peptide, backpack sling sprite edited by Boaz1111, wield sprites by RiceMar1244", | ||
"size": { | ||
"x": 32, | ||
"y": 32 | ||
|
@@ -27,11 +27,19 @@ | |
"name": "inhand-right", | ||
"directions": 4 | ||
}, | ||
{ | ||
"name": "wielded-inhand-left", | ||
"directions": 4 | ||
}, | ||
{ | ||
"name": "wielded-inhand-right", | ||
"directions": 4 | ||
}, | ||
Comment on lines
+30
to
+37
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧹 Nitpick (assertive) Новые состояния для двуручного оружия одобрены. Добавление состояний "wielded-inhand-left" и "wielded-inhand-right" соответствует целям PR по внедрению функциональности двойного владения оружием. Количество направлений (4) соответствует существующим состояниям "inhand", что обеспечивает согласованность. Предлагаю небольшое улучшение в организации кода: переместите новые состояния сразу после соответствующих "inhand" состояний для лучшей читаемости и группировки связанных элементов. Например: {
"name": "inhand-left",
"directions": 4
},
{
"name": "wielded-inhand-left",
"directions": 4
},
{
"name": "inhand-right",
"directions": 4
},
{
"name": "wielded-inhand-right",
"directions": 4
}, |
||
{ | ||
"name": "equipped-BACKPACK", | ||
"directions": 4 | ||
}, | ||
{ | ||
{ | ||
"name": "equipped-SUITSTORAGE", | ||
"directions": 4 | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧹 Nitpick (assertive)
Изменения соответствуют целям PR и расширяют функциональность спрайтов.
Добавление новых состояний "equipped-SUITSTORAGE", "wielded-inhand-left" и "wielded-inhand-right" расширяет визуальное представление винтовки в игре, что соответствует цели добавления функциональности двойного владения оружием. Изменения выглядят корректными и последовательными.
Для улучшения читаемости и организации кода, рекомендуется сгруппировать похожие состояния вместе. Например, можно расположить все состояния "inhand" рядом друг с другом, а все состояния "equipped" - в другой группе. Это может упростить будущее обслуживание и понимание структуры файла.