Halloween things #289
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,32 @@ | ||
| ent-LPPCandy = конфета | ||
| .desc = Обычная конфета состоящия из сахара и других вещей. | ||
| ent-LPPCandy2 = конфета | ||
| .desc = Обычная конфета состоящия из сахара и других вещей. | ||
| ent-LPPCandy3 = конфета | ||
| .desc = Обычная конфета состоящия из сахара и других вещей. | ||
| ent-LPPCandy4 = конфета | ||
| .desc = Обычная конфета состоящия из сахара и других вещей. | ||
| ent-LPPLolipop = конфета на палочке | ||
| .desc = Всё такая же конфета, но на палочке. | ||
|
There was a problem hiding this comment. Исправьте капитализацию Согласно правилам кодирования, перевод, начинающийся с "ent-", должен иметь заглавную букву после знака "=". Исправьте это следующим образом: |
||
| ent-HalloweenTree01 = мёртвое дерево | ||
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | ||
| ent-HalloweenTree02 = мёртвое дерево | ||
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | ||
| ent-HalloweenTree03 = мёртвое дерево | ||
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | ||
| ent-HalloweenTree04 = мёртвое дерево | ||
| .desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко. | ||
|
There was a problem hiding this comment. 🛠️ Refactor suggestion Оптимизируйте повторяющиеся переводы Для повторяющихся переводов используйте следующую структуру: Это сделает код более поддерживаемым и уменьшит дублирование. |
||
| ent-LPPPSpiderDeco = паучки | ||
| .desc = Муляжи паучков которые выглядят так, будто готовы вас покусать. | ||
|
There was a problem hiding this comment. Исправьте капитализацию Согласно правилам кодирования, перевод, начинающийся с "ent-", должен иметь заглавную букву после знака "=". Исправьте это следующим образом: Также добавлена запятая после "паучков" для улучшения читаемости. |
||
| ent-LPPPaperBagCandy = бумажный пакет | ||
| .desc = Пакет из бумаги, украшен в тематике Хэллоиина. | ||
|
There was a problem hiding this comment. Исправьте капитализацию и орфографию
Исправленный вариант: |
||
| ent-LPPPumpkinBagCandy = корзинка из тыквы | ||
| .desc = Корзинка вырезанная из тыквы и приделанной ручкой. | ||
|
There was a problem hiding this comment. Исправьте капитализацию и грамматику
Исправленный вариант: |
||
| ent-LPPPosterHalloweenFamily = призрачная семья | ||
| .desc = Картина семьи призраков, они выглядат счастливыми. | ||
|
There was a problem hiding this comment. Исправьте капитализацию и орфографию
Исправленный вариант: |
||
| ent-LPPPosterHalloweencounts = картина графа | ||
| .desc = Картина с каким-то вампиром, от его взгляда вам становиться холодно. | ||
|
There was a problem hiding this comment. Исправьте капитализацию и орфографию
Исправленный вариант: |
||
| ent-LPPPCheckboxes = флажки | ||
| .desc = Чёрно-оранжевые флажки, навевают чуством Хэллоиина. | ||
|
There was a problem hiding this comment. Исправьте капитализацию и орфографию
Исправленный вариант: |
||
| ent-LPPPLightbulbs = гирлянда | ||
| .desc = подвесная гирлянда, не самый лучший источник освещения. | ||
|
There was a problem hiding this comment. Исправьте капитализацию Согласно правилам кодирования, перевод, начинающийся с "ent-", должен иметь заглавную букву после знака "=". Также, для соблюдения единообразия, начните описание с заглавной буквы. Исправленный вариант: There was a problem hiding this comment. 🛠️ Refactor suggestion Общие рекомендации по улучшению локализации
Применение этих рекомендаций значительно повысит качество и поддерживаемость локализации. |
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,75 @@ | ||
| - type: entity | ||
| name: candy | ||
| parent: FoodBase | ||
| id: LPPCandy | ||
| description: This is my kingdom come, this is my kingdom come... | ||
| components: | ||
| - type: Sprite | ||
| sprite: _LostParadise/Objects/Consumable/halloween.rsi | ||
| state: candy_1 | ||
| - type: Item | ||
| size: Tiny | ||
| shape: | ||
| - 0, 0, 0, 0 | ||
| - type: SolutionContainerManager | ||
| solutions: | ||
| food: | ||
| maxVol: 5 | ||
| reagents: | ||
| - ReagentId: Sugar | ||
| Quantity: 3 | ||
| - ReagentId: Nutriment | ||
| Quantity: 1 | ||
| - ReagentId: Theobromine | ||
| Quantity: 1 | ||
| - type: Tag | ||
| tags: | ||
| - LPPCandy | ||
|
Comment on lines
+1
to
+27
There was a problem hiding this comment. 🧹 Nitpick (assertive) Необходимо изменить описание и исправить отступы
Предлагаемые изменения для описания: - description: This is my kingdom come, this is my kingdom come...
+ description: A sweet treat perfect for Halloween.
🧰 Tools🪛 yamllint
|
||
|
|
||
| - type: entity | ||
| parent: LPPCandy | ||
| id: LPPCandy2 | ||
| components: | ||
| - type: Sprite | ||
| state: candy_2 | ||
|
Comment on lines
+29
to
+34
There was a problem hiding this comment. 🧹 Nitpick (assertive) Исправьте отступ в компоненте Sprite Отступ в строке 33 должен быть увеличен до 4 пробелов для соответствия стилю остального кода. 🧰 Tools🪛 yamllint
|
||
|
|
||
| - type: entity | ||
| parent: LPPCandy | ||
| id: LPPCandy3 | ||
| components: | ||
| - type: Sprite | ||
| state: candy_3 | ||
|
Comment on lines
+36
to
+41
There was a problem hiding this comment. 🧹 Nitpick (assertive) Исправьте отступ в компоненте Sprite Отступ в строке 40 должен быть увеличен до 4 пробелов для соответствия стилю остального кода. 🧰 Tools🪛 yamllint
|
||
|
|
||
| - type: entity | ||
| parent: LPPCandy | ||
| id: LPPCandy4 | ||
| components: | ||
| - type: Sprite | ||
| state: candy_4 | ||
|
Comment on lines
+43
to
+48
There was a problem hiding this comment. 🧹 Nitpick (assertive) Исправьте отступ в компоненте Sprite Отступ в строке 47 должен быть увеличен до 4 пробелов для соответствия стилю остального кода. 🧰 Tools🪛 yamllint
|
||
|
|
||
| - type: entity | ||
| name: lolipop | ||
| parent: LPPCandy | ||
| id: LPPLolipop | ||
| description: This is my kingdom come, this is my kingdom come... | ||
| components: | ||
| - type: Sprite | ||
| state: lollipop | ||
| - type: Item | ||
| size: Tiny | ||
| shape: | ||
| - 0, 0, 0, 1 | ||
| - type: SolutionContainerManager | ||
| solutions: | ||
| food: | ||
| maxVol: 15 | ||
| reagents: | ||
| - ReagentId: Sugar | ||
| Quantity: 8 | ||
| - ReagentId: Nutriment | ||
| Quantity: 4 | ||
| - ReagentId: Theobromine | ||
| Quantity: 3 | ||
| - type: Tag | ||
| tags: | ||
| - LPPCandy | ||
|
Comment on lines
+50
to
+75
There was a problem hiding this comment. 🧹 Nitpick (assertive) Необходимо обновить описание, исправить отступы и добавить новую строку в конце файла
Предлагаемые изменения для описания: - description: This is my kingdom come, this is my kingdom come...
+ description: A colorful lollipop, perfect for trick-or-treating.
🧰 Tools🪛 yamllint
|
||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,40 @@ | ||||||||||||||||||||||||||||||||||||||
| - type: entity | ||||||||||||||||||||||||||||||||||||||
| parent: BaseTree | ||||||||||||||||||||||||||||||||||||||
| id: HalloweenTree01 | ||||||||||||||||||||||||||||||||||||||
| name: dead tree | ||||||||||||||||||||||||||||||||||||||
| description: Dead tree with leaves on it, looks creepy. | ||||||||||||||||||||||||||||||||||||||
| components: | ||||||||||||||||||||||||||||||||||||||
| - type: Sprite | ||||||||||||||||||||||||||||||||||||||
| sprite: _LostParadise/Objects/Decoration/Flora/flora_halloween.rsi | ||||||||||||||||||||||||||||||||||||||
| state: halloweentree01 | ||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+1
to
+9
There was a problem hiding this comment. Определение сущности корректно, но есть проблема с отступами. Определение сущности Однако, есть проблема с отступами в компоненте Sprite. Исправьте отступ следующим образом: components:
- - type: Sprite
+ - type: Sprite
sprite: _LostParadise/Objects/Decoration/Flora/flora_halloween.rsi
state: halloweentree01
🧰 Tools🪛 yamllint
|
||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
| - type: entity | ||||||||||||||||||||||||||||||||||||||
| parent: BaseTree | ||||||||||||||||||||||||||||||||||||||
| id: HalloweenTree02 | ||||||||||||||||||||||||||||||||||||||
| name: dead tree | ||||||||||||||||||||||||||||||||||||||
| description: Dead tree with leaves on it, looks creepy. | ||||||||||||||||||||||||||||||||||||||
| components: | ||||||||||||||||||||||||||||||||||||||
| - type: Sprite | ||||||||||||||||||||||||||||||||||||||
| sprite: _LostParadise/Objects/Decoration/Flora/flora_halloween.rsi | ||||||||||||||||||||||||||||||||||||||
| state: halloweentree02 | ||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+11
to
+19
There was a problem hiding this comment. Определение сущности корректно, но присутствует та же проблема с отступами. Определение сущности Исправьте отступ в компоненте Sprite аналогично предыдущему комментарию: components:
- - type: Sprite
+ - type: Sprite
sprite: _LostParadise/Objects/Decoration/Flora/flora_halloween.rsi
state: halloweentree02📝 Committable suggestion
Suggested change
🧰 Tools🪛 yamllint
|
||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
| - type: entity | ||||||||||||||||||||||||||||||||||||||
| parent: BaseTree | ||||||||||||||||||||||||||||||||||||||
| id: HalloweenTree03 | ||||||||||||||||||||||||||||||||||||||
| name: dead tree | ||||||||||||||||||||||||||||||||||||||
| description: Dead tree with leaves on it, looks creepy. | ||||||||||||||||||||||||||||||||||||||
| components: | ||||||||||||||||||||||||||||||||||||||
| - type: Sprite | ||||||||||||||||||||||||||||||||||||||
| sprite: _LostParadise/Objects/Decoration/Flora/flora_halloween.rsi | ||||||||||||||||||||||||||||||||||||||
| state: halloweentree03 | ||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+21
to
+29
There was a problem hiding this comment. Определение сущности корректно, но сохраняется проблема с отступами. Определение сущности Пожалуйста, исправьте отступ в компоненте Sprite: components:
- - type: Sprite
+ - type: Sprite
sprite: _LostParadise/Objects/Decoration/Flora/flora_halloween.rsi
state: halloweentree03📝 Committable suggestion
Suggested change
🧰 Tools🪛 yamllint
|
||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
| - type: entity | ||||||||||||||||||||||||||||||||||||||
| parent: BaseTree | ||||||||||||||||||||||||||||||||||||||
| id: HalloweenTree04 | ||||||||||||||||||||||||||||||||||||||
| name: dead tree | ||||||||||||||||||||||||||||||||||||||
| description: Dead tree with leaves on it, looks creepy. | ||||||||||||||||||||||||||||||||||||||
| components: | ||||||||||||||||||||||||||||||||||||||
| - type: Sprite | ||||||||||||||||||||||||||||||||||||||
| sprite: _LostParadise/Objects/Decoration/Flora/flora_halloween.rsi | ||||||||||||||||||||||||||||||||||||||
| state: halloweentree04 | ||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+1
to
+40
There was a problem hiding this comment. Общий обзор файла Файл содержит корректные определения четырех сущностей деревьев для Хэллоуина. Все сущности правильно размещены в папке _LostParadise, и в файле отсутствуют русские переводы, что соответствует указанным рекомендациям. Однако, есть несколько проблем с форматированием:
Рекомендуется исправить эти проблемы форматирования. Вот команда sed, которая может автоматически исправить отступы во всем файле: sed -i 's/^ - type: Sprite/ - type: Sprite/' Resources/Prototypes/_LostParadise/Entities/Objects/Decoration/Flora/halloween.ymlТакже не забудьте удалить лишнюю пустую строку в конце файла. 🧰 Tools🪛 yamllint
Comment on lines
+31
to
+40
There was a problem hiding this comment. Определение сущности корректно, но есть проблемы с форматированием. Определение сущности Пожалуйста, внесите следующие изменения:
components:
- - type: Sprite
+ - type: Sprite
sprite: _LostParadise/Objects/Decoration/Flora/flora_halloween.rsi
state: halloweentree04
🧰 Tools🪛 yamllint
|
||||||||||||||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,34 @@ | ||
| - type: entity | ||
| parent: BaseSign | ||
| id: LPPPSpiderDeco | ||
| name: spiders | ||
| description: Fake spiders that look like they're ready to bite you. | ||
| components: | ||
| - type: Sprite | ||
| sprite: _LostParadise/Objects/Decoration/halloween.rsi | ||
| snapCardinals: true | ||
| layers: | ||
| - state: spider_big | ||
| map: [ "enum.DamageStateVisualLayers.Base" ] | ||
|
Comment on lines
+7
to
+12
There was a problem hiding this comment. 🧹 Nitpick (assertive) Компонент Sprite определен корректно. Путь к спрайту и определение слоя соответствуют требованиям. Используется правильная папка _LostParadise. Однако, обратите внимание на лишние пробелы внутри квадратных скобок в строке 12: map: [ "enum.DamageStateVisualLayers.Base" ]Рекомендуется удалить лишние пробелы: map: ["enum.DamageStateVisualLayers.Base"]🧰 Tools🪛 yamllint
|
||
| - type: InteractionOutline | ||
| - type: RandomSprite | ||
| available: | ||
| - enum.DamageStateVisualLayers.Base: | ||
| spider_big: "" | ||
| spider_big2: "" | ||
| spider_mini1: "" | ||
| spider_mini2: "" | ||
| spider_mini3: "" | ||
| - type: Damageable | ||
| damageContainer: Inorganic | ||
| - type: Destructible | ||
| thresholds: | ||
| - trigger: | ||
| !type:DamageTrigger | ||
| damage: 10 | ||
| behaviors: | ||
| - !type:PlaySoundBehavior | ||
| sound: | ||
| collection: WoodDestroyHeavy | ||
| - !type:DoActsBehavior | ||
| acts: [ "Destruction" ] | ||
|
Comment on lines
+24
to
+34
There was a problem hiding this comment. 🧹 Nitpick (assertive) Компонент Destructible реализован корректно, но есть предложения по улучшению. Компонент Destructible хорошо определен и подходит для разрушаемого декоративного объекта. Однако есть два момента, на которые стоит обратить внимание:
acts: [ "Destruction" ]Рекомендуется исправить на: acts: ["Destruction"]🧰 Tools🪛 yamllint
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,35 @@ | ||
| - type: entity | ||
| name: paper bag | ||
| id: LPPPaperBagCandy | ||
| parent: CandyBucket | ||
| description: A bag for storing chemistry products, such as pills, pill canisters, bottles, and syringes. | ||
| components: | ||
| - type: Sprite | ||
| sprite: _LostParadise/Objects/Specific/halloween/paper_bag.rsi | ||
| layers: | ||
| - state: empty_icon | ||
| map: [ "enum.ToggleVisuals.Layer" ] | ||
| - type: Storage | ||
| grid: | ||
| - 0,0,4,2 | ||
| whitelist: | ||
| tags: | ||
| - LPPCandy | ||
|
Comment on lines
+1
to
+17
There was a problem hiding this comment. 🧹 Nitpick (assertive) Необходимо внести изменения в определение сущности "paper bag"
🧰 Tools🪛 yamllint
|
||
|
|
||
| - type: entity | ||
| name: pumpkin bag | ||
| id: LPPPumpkinBagCandy | ||
| parent: CandyBucket | ||
| description: A bag for storing chemistry products, such as pills, pill canisters, bottles, and syringes. | ||
| components: | ||
| - type: Sprite | ||
| sprite: _LostParadise/Objects/Specific/halloween/pumpkin_bag.rsi | ||
| layers: | ||
| - state: empty_icon | ||
| map: [ "enum.ToggleVisuals.Layer" ] | ||
| - type: Storage | ||
| grid: | ||
| - 0,0,4,2 | ||
| whitelist: | ||
| tags: | ||
| - LPPCandy | ||
|
Comment on lines
+19
to
+35
There was a problem hiding this comment. 🧹 Nitpick (assertive) Необходимо внести изменения в определение сущности "pumpkin bag" и исправить форматирование файла
🧰 Tools🪛 yamllint
|
||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,47 @@ | ||||||||||||||||||||||||||||||||||||||||||
| - type: entity | ||||||||||||||||||||||||||||||||||||||||||
| parent: LPPPosterBase | ||||||||||||||||||||||||||||||||||||||||||
| id: LPPPosterHalloweenFamily | ||||||||||||||||||||||||||||||||||||||||||
| name: ghoast family | ||||||||||||||||||||||||||||||||||||||||||
| description: Picture of a family of ghosts, they look happy. | ||||||||||||||||||||||||||||||||||||||||||
| components: | ||||||||||||||||||||||||||||||||||||||||||
| - type: Sprite | ||||||||||||||||||||||||||||||||||||||||||
| sprite: _LostParadise/Structures/Wallmounts/halloween.rsi | ||||||||||||||||||||||||||||||||||||||||||
| state: picture_family | ||||||||||||||||||||||||||||||||||||||||||
| noRot: true | ||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||
| - type: entity | ||||||||||||||||||||||||||||||||||||||||||
| parent: LPPPosterBase | ||||||||||||||||||||||||||||||||||||||||||
| id: LPPPosterHalloweencounts | ||||||||||||||||||||||||||||||||||||||||||
| name: count's painting | ||||||||||||||||||||||||||||||||||||||||||
| description: Picture of some kind of vampire, the sight of him makes you feel cold. | ||||||||||||||||||||||||||||||||||||||||||
| components: | ||||||||||||||||||||||||||||||||||||||||||
| - type: Sprite | ||||||||||||||||||||||||||||||||||||||||||
| sprite: _LostParadise/Structures/Wallmounts/halloween.rsi | ||||||||||||||||||||||||||||||||||||||||||
| state: сounts_painting | ||||||||||||||||||||||||||||||||||||||||||
| noRot: true | ||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+12
to
+21
There was a problem hiding this comment. Исправьте опечатку в имени состояния спрайта. Определение сущности Пожалуйста, примените следующее исправление: - state: сounts_painting
+ state: counts_paintingЭто заменит кириллическую 'с' на латинскую 'c' в имени состояния спрайта, что предотвратит возможные проблемы при обращении к спрайту. 📝 Committable suggestion
Suggested change
🧰 Tools🪛 yamllint
|
||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||
| - type: entity | ||||||||||||||||||||||||||||||||||||||||||
| parent: BaseSign | ||||||||||||||||||||||||||||||||||||||||||
| id: LPPPCheckboxes | ||||||||||||||||||||||||||||||||||||||||||
| name: checkboxes | ||||||||||||||||||||||||||||||||||||||||||
| description: Black and orange flags, it's got a halloween feel to it. | ||||||||||||||||||||||||||||||||||||||||||
| components: | ||||||||||||||||||||||||||||||||||||||||||
| - type: Sprite | ||||||||||||||||||||||||||||||||||||||||||
| sprite: _LostParadise/Structures/Wallmounts/halloween.rsi | ||||||||||||||||||||||||||||||||||||||||||
| state: checkboxes | ||||||||||||||||||||||||||||||||||||||||||
| noRot: true | ||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+23
to
+32
There was a problem hiding this comment. 🧹 Nitpick (assertive) Удалите лишний пробел в конце описания. Определение сущности Пожалуйста, примените следующее исправление: - description: Black and orange flags, it's got a halloween feel to it.
+ description: Black and orange flags, it's got a halloween feel to it.Это удалит лишний пробел в конце строки описания, что улучшит форматирование файла.
🧰 Tools🪛 yamllint
|
||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||
| - type: entity | ||||||||||||||||||||||||||||||||||||||||||
| parent: BaseSign | ||||||||||||||||||||||||||||||||||||||||||
| id: LPPPLightbulbs | ||||||||||||||||||||||||||||||||||||||||||
| name: light bulbs | ||||||||||||||||||||||||||||||||||||||||||
| description: Hanging garland, not the best source of light | ||||||||||||||||||||||||||||||||||||||||||
| components: | ||||||||||||||||||||||||||||||||||||||||||
| - type: PointLight | ||||||||||||||||||||||||||||||||||||||||||
| radius: 2.0 | ||||||||||||||||||||||||||||||||||||||||||
| energy: 2.5 | ||||||||||||||||||||||||||||||||||||||||||
| color: "#e8b964" | ||||||||||||||||||||||||||||||||||||||||||
| - type: Sprite | ||||||||||||||||||||||||||||||||||||||||||
| sprite: _LostParadise/Structures/Wallmounts/halloween.rsi | ||||||||||||||||||||||||||||||||||||||||||
| state: light_bulbs | ||||||||||||||||||||||||||||||||||||||||||
| noRot: true | ||||||||||||||||||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,26 @@ | ||
| { | ||
| "version": 1, | ||
| "license": "CC-BY-SA-3.0", | ||
| "copyright": "Taken from tgstation at commit https://github.com/tgstation/tgstation/commit/e00cae8d065f9cf520688cc0dd0e15ba5bef12a9", | ||
|
Comment on lines
+1
to
+4
There was a problem hiding this comment. 🧹 Nitpick (assertive) Метаданные корректны, но можно улучшить. Информация о версии, лицензии и авторских правах указана правильно. Хорошо, что вы включили ссылку на конкретный коммит. Рекомендую добавить дату коммита для облегчения отслеживания изменений в будущем. Например: - "copyright": "Taken from tgstation at commit https://github.com/tgstation/tgstation/commit/e00cae8d065f9cf520688cc0dd0e15ba5bef12a9",
+ "copyright": "Taken from tgstation at commit https://github.com/tgstation/tgstation/commit/e00cae8d065f9cf520688cc0dd0e15ba5bef12a9 (Date: YYYY-MM-DD)",
|
||
| "size": { | ||
| "x": 32, | ||
| "y": 32 | ||
| }, | ||
| "states": [ | ||
| { | ||
| "name": "candy_1" | ||
| }, | ||
| { | ||
| "name": "candy_2" | ||
| }, | ||
| { | ||
| "name": "candy_3" | ||
| }, | ||
| { | ||
| "name": "candy_4" | ||
| }, | ||
| { | ||
| "name": "lollipop" | ||
| } | ||
| ] | ||
|
Comment on lines
+9
to
+25
There was a problem hiding this comment. 🧹 Nitpick (assertive) Состояния определены корректно, но можно улучшить описание. Пять различных состояний для конфет обеспечивают хорошее разнообразие. Названия понятны и последовательны. Для улучшения ясности рекомендую добавить краткие описания для каждого типа конфет. Например: {
"name": "candy_1"
+ "description": "Круглая конфета в красной обертке"
},
{
"name": "candy_2"
+ "description": "Овальная конфета в синей обертке"
},Это поможет разработчикам и дизайнерам лучше понимать визуальные различия между типами конфет.
|
||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,26 @@ | ||
| { | ||
| "version": 1, | ||
| "license": "CC-BY-SA-3.0", | ||
| "copyright": "Taken from tgstation at commit https://github.com/tgstation/tgstation/commit/e00cae8d065f9cf520688cc0dd0e15ba5bef12a9", | ||
|
Comment on lines
+1
to
+4
There was a problem hiding this comment. 🧹 Nitpick (assertive) Информация о версии и лицензии корректна. Версия, лицензия и информация об авторских правах указаны правильно. Хорошая практика включать ссылку на исходный репозиторий и хеш коммита. Рекомендуется добавить дату коммита для облегчения отслеживания изменений в будущем. |
||
| "size": { | ||
| "x": 64, | ||
| "y": 64 | ||
| }, | ||
| "states": [ | ||
| { | ||
| "name": "icon" | ||
| }, | ||
| { | ||
| "name": "halloweentree01" | ||
| }, | ||
| { | ||
| "name": "halloweentree02" | ||
| }, | ||
| { | ||
| "name": "halloweentree03" | ||
| }, | ||
| { | ||
| "name": "halloweentree04" | ||
| } | ||
| ] | ||
|
Comment on lines
+9
to
+25
There was a problem hiding this comment. 🧹 Nitpick (assertive) Состояния текстуры определены корректно. Пять состояний (включая "icon" и четыре варианта хэллоуинских деревьев) определены правильно. Структура JSON и именование состояний соответствуют стандартам. Для улучшения читаемости и понимания, рекомендуется добавить краткие комментарии к каждому состоянию, описывающие их визуальные отличия. |
||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,26 @@ | ||
| { | ||
| "version": 1, | ||
| "license": "CC-BY-SA-3.0", | ||
| "copyright": "Sprited", | ||
| "size": { | ||
| "x": 32, | ||
| "y": 32 | ||
| }, | ||
| "states": [ | ||
| { | ||
| "name": "spider_big" | ||
| }, | ||
| { | ||
| "name": "spider_big2" | ||
| }, | ||
| { | ||
| "name": "spider_mini1" | ||
| }, | ||
| { | ||
| "name": "spider_mini2" | ||
| }, | ||
| { | ||
| "name": "spider_mini3" | ||
| } | ||
| ] | ||
|
Comment on lines
+9
to
+25
There was a problem hiding this comment. 🛠️ Refactor suggestion Рекомендуется добавить дополнительные метаданные для состояний. Текущие состояния определены корректно, но можно улучшить их описание: Предлагаемые изменения: "states": [
{
- "name": "spider_big"
+ "name": "spider_big",
+ "delays": [[1.0]],
+ "directions": 1
},
{
- "name": "spider_big2"
+ "name": "spider_big2",
+ "delays": [[1.0]],
+ "directions": 1
},
// Аналогично для остальных состояний
]Это поможет:
|
||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,33 @@ | ||
| { | ||
| "version": 1, | ||
| "license": "CC-BY-SA-3.0", | ||
| "copyright": "Sprited", | ||
|
There was a problem hiding this comment. Требуется уточнение авторства в поле copyright. Значение "Sprited" недостаточно информативно. Рекомендуется указать конкретного автора или команду, создавшую спрайты. |
||
| "size": { | ||
| "x": 32, | ||
| "y": 32 | ||
| }, | ||
| "states": [ | ||
| { | ||
| "name": "empty_icon" | ||
| }, | ||
| { | ||
| "name": "full_icon" | ||
| }, | ||
| { | ||
| "name": "empty-inhand-right", | ||
| "directions": 4 | ||
| }, | ||
| { | ||
| "name": "empty-inhand-left", | ||
| "directions": 4 | ||
| }, | ||
| { | ||
| "name": "full-inhand-right", | ||
| "directions": 4 | ||
| }, | ||
| { | ||
| "name": "full-inhand-left", | ||
| "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.
Исправьте форматирование и повторяющиеся переводы
Согласно правилам кодирования, все переводы, начинающиеся с "ent-", должны иметь заглавную букву после знака "=". Исправьте это для всех записей конфет.
Для повторяющихся переводов используйте следующую структуру: