TGUI для консоли генетика by TEXH4Pb · Pull Request #14267 · TauCetiStation/TauCetiClassic

TEXH4Pb · 2025-07-22T20:39:04Z

Описание изменений

Расписывать нечего, скрины ниже. Может быть попутно случайно или специально пофиксил какие-то мелкие баги, но я не помню.

Скрины

1

2

3

4

5

6

7

8

Ещё нужно поправить некоторые мелочи, но закинул в виде драфта. Пусть будет, может кто-то накинет толкового фидбека по поводу внешнего вида интерфейса или кодстайла. По последнему вообще очень нужны дельные замечания, т.к. читаемость кода в js файле оставляет желать лучшего.

Почему и что этот ПР улучшит

Ещё одним nanoUI интерфейсом меньше.

Авторство

Я

Чеинжлог

🆑 TEXHAPb

tweak: консоль генетика получила новый GUI

TauKitty · 2025-07-22T20:39:08Z

Changelog status: ✔️

TEXH4Pb · 2025-07-22T20:48:23Z

Есть беда, что диммер не растягивается на всю высоту окна, когда оно "прокручено" вниз скроллбаром. Остаётся незакрытый огрызок. Как пофиксить- не знаю. Походу проблема самого диммера. Очень нужна помощь.

RichardJones1 · 2025-08-10T13:09:02Z

Есть беда, что диммер не растягивается на всю высоту окна, когда оно "прокручено" вниз скроллбаром. Остаётся незакрытый огрызок. Как пофиксить- не знаю. Походу проблема самого диммера. Очень нужна помощь.

Надо наверное глянуть, если подобное на других билдах. Мб уже в core tgui это пофиксили.

TEXH4Pb · 2025-08-17T13:37:31Z

Есть беда, что диммер не растягивается на всю высоту окна, когда оно "прокручено" вниз скроллбаром. Остаётся незакрытый огрызок. Как пофиксить- не знаю. Походу проблема самого диммера. Очень нужна помощь.

Надо наверное глянуть, если подобное на других билдах. Мб уже в core tgui это пофиксили.

Свежий tgui работает на React'е, поэтому там диммер напрямую из него используется. Хз, где ещё стоит искать, кроме самого tgstation.

DarthSidiousPalpatine · 2025-08-21T13:29:02Z

Очень нужна помощь.

Ну я могу только предложить костыль который мб сработает.
Блоку с буфферами отдельно сделай прокрутку, чтобы только блок крутился, а окно не крутилось...

Хз поможет ли.

TEXH4Pb · 2025-08-25T21:05:37Z

Ура, я починил!

TEXH4Pb · 2025-08-25T21:06:43Z

Ну я могу только предложить костыль который мб сработает. Блоку с буфферами отдельно сделай прокрутку, чтобы только блок крутился, а окно не крутилось...

Хз поможет ли.

Хорошая идея, спасибо. Но не сработала. Секциям scrollable почему-то просто добавляет неактивный скролл-бар, который никогда и ни при каких обстоятельствах не активируется.

TEXH4Pb · 2025-08-25T21:09:38Z

Я балда, короче, надо было диммер запихивать внутрь Window, а не в Window.Content.

TEXH4Pb · 2025-08-25T21:25:20Z

Ну всё, пора

TEXH4Pb · 2025-08-25T21:34:20Z

Я не понимаю, почему дифф такой огромный, я не так много менял в dna_modifier.dm 😕

AirBlack · 2025-08-26T16:12:55Z

tgui/packages/tgui/interfaces/DnaModifier.js

+  const {
+    selectedMenuKey,
+    hasOccupant,
+    isInjectorReady,
+    hasDisk,
+    disk,
+    buffers,
+    radiationIntensity,
+    radiationDuration,
+    irradiating,
+    dnaBlockSize,
+    selectedUIBlock,
+    selectedUISubBlock,
+    selectedSEBlock,
+    selectedSESubBlock,
+    selectedUITarget,
+    selectedUITargetHex,
+    occupant,
+    isBeakerLoaded,
+    beakerLabel,
+    beakerVolume,
+    beakerMaxVolume,
+    injectAmount,
+  } = data;


слишком дофига данных, разделил бы хоть на подобъедки

Приведи пример, пожалуйста, я не очень силён в работе с данными в tgui.

со стороны бульонда

dmTguiList["selectedSE"] = new list( "block" = selectedSEBlock, "subBlock" = selectedSESubBlock, )

со стороны тгуи

const { selectedSE } = data; console.log(selectedSE.block);

глянь пример в CrewMonitor, я там на под объедки делил

AirBlack · 2025-08-26T16:13:37Z

слишком большой файл и слишком здоровый switch, мог бы отдельные табы как отдельные компоненты сделать

AirBlack · 2025-08-26T16:19:20Z

code/game/dna/dna_modifier.dm

-			ui.set_initial_data(data)
-			// open the new ui window
+
+		ui = SStgui.try_update_ui(user, src, ui)


проверки перенести в tgui_state

AirBlack · 2025-08-26T16:20:23Z

code/game/dna/dna_modifier.dm

+			if (selected_ui_target < 16 && selected_ui_target >= 0)
+				switch(selected_ui_target)
+					if(10)
+						selected_ui_target_hex = "A"
+					if(11)
+						selected_ui_target_hex = "B"
+					if(12)
+						selected_ui_target_hex = "C"
+					if(13)
+						selected_ui_target_hex = "D"
+					if(14)
+						selected_ui_target_hex = "E"
+					if(15)
+						selected_ui_target_hex = "F"
+					else
+						selected_ui_target_hex = selected_ui_target


а нельзя со стороны клиента уже цифру в виде hex отправлять?

Перенастроить NumberInput, чтобы он сразу возвращал hex, нельзя (или я не знаю способа). Только писать новый UI, элемент, чего я делать не буду. А просто конвертировать через такой же страшный свитч можно, наверное, но я не понимаю, почему этим должен клиент заниматься.

Чтобы особенности реализации интерфейса остались в коде самого интерфейса, а код бульондовский об интерфейсе ничего не знал кроме как инпутов аутпутов, разделение бизнес логики от логики отображения, умные слова бла бла бла, имхо конвертация это ответственность клиента, ну лан пох

AirBlack · 2025-08-26T16:21:43Z

tgui/packages/tgui/interfaces/DnaModifier.js

+                {!!disk.data && !!hasDisk && (
+                  <LabeledList.Item label="Label">
+                    {disk.label}
+                  </LabeledList.Item>
+                )}
+                {!!disk.data && !!hasDisk && (
+                  <LabeledList.Item label="Subject">
+                    {disk.owner ? disk.owner : 'Unknown'}
+                  </LabeledList.Item>
+                )}
+                {!!disk.data && !!hasDisk && (
+                  <LabeledList.Item label="Stored data">
+                    {disk.type === 'ui'
+                      ? 'Unique identifier'
+                      : 'Structural enzymes'}
+                    {disk.ue ? ' + unique enzymes' : ''}
+                  </LabeledList.Item>
+                )}


три одинаковых проверки на disk.data и hasDisk

KIBORG04 · 2025-09-05T18:54:03Z

доверяю аирблеку, делой

KIBORG04 · 2025-09-05T18:57:25Z

code/game/dna/dna_modifier.dm

+			irradiating = radiation_duration
+			var/lock_state = connected.locked
+			connected.locked = TRUE//lock it
+			SStgui.try_update_ui(ui.user, src, ui) // update all UIs attached to src


По дефолту autoupdate у тгуи включено, я думаю это не нужно тут

KIBORG04 · 2025-09-05T18:57:35Z

code/game/dna/dna_modifier.dm

+			SStgui.try_update_ui(ui.user, src, ui) // update all UIs attached to src

-		sleep(radiation_duration SECONDS)
+			sleep(radiation_duration SECONDS)


Было бы круче без слипа тут, но ок

Это старый код, который я особо не трогал, просто переносил в другие места. Спустя два месяца вникать заново и думать над его рефактором мне, мягко говоря, не охота.

AirBlack · 2025-09-06T08:50:45Z

доверяю аирблеку, делой

я так, краем глаза прошелся, не вчитывался не полноценно ревувал короч

TEXH4Pb · 2025-09-16T09:36:14Z

Я когда-нибудь потом займусь, пока не закрывайте.

github-actions · 2025-10-10T12:41:38Z

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений и обсуждения. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Попробуйте связаться с мейнтейнерами, если они по какой-то причине игнорируют ПР, или у вас есть вопросы. Спасибо за ваш вклад.

github-actions · 2025-10-10T12:41:38Z

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений и обсуждения. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Попробуйте связаться с мейнтейнерами, если они по какой-то причине игнорируют ПР, или у вас есть вопросы. Спасибо за ваш вклад.

github-actions · 2025-10-17T14:16:44Z

ПР закрыт из-за длительного отсуствия активности. Для переоткрытия ПРа, пожалуйста, обратитесь к кому-либо из мейнтейнеров. Вы можете призвать их в комментарии слапнув @TauCetiStation/maintainers.

github-actions · 2025-11-11T02:57:09Z

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений и обсуждения. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Попробуйте связаться с мейнтейнерами, если они по какой-то причине игнорируют ПР, или у вас есть вопросы. Спасибо за ваш вклад.

Genetics TGUI

1de48c1

TauKitty added Tweak UI User Interface labels Jul 22, 2025

TEXH4Pb marked this pull request as draft July 22, 2025 20:39

TEXH4Pb changed the title ~~[DNM] TGUI для консоли генетика~~ TGUI для консоли генетика Jul 22, 2025

TEXH4Pb added 2 commits July 22, 2025 23:42

Merge remote-tracking branch 'upstream/master' into genetics-tgui

54a2211

Update tgui.bundle.js

886e4a1

AzzyDreemurr13 added this to the NanoUI/BrowserUI --> TGUI milestone Jul 22, 2025

AzzyDreemurr13 added the Help Wanted label Jul 22, 2025

TEXH4Pb added 2 commits August 17, 2025 16:18

Merge remote-tracking branch 'upstream/master' into genetics-tgui

8b2cc96

Update tgui.bundle.js

f4daf18

TEXH4Pb added 2 commits August 26, 2025 00:15

Dimmer fix

3e4a7f5

emitter defines

881510c

TEXH4Pb marked this pull request as ready for review August 25, 2025 21:26

TEXH4Pb added 2 commits August 26, 2025 00:26

Merge remote-tracking branch 'upstream/master' into genetics-tgui

b7d39a7

Update tgui.bundle.js

8e09391

Update dna_modifier.dm

bae08ff

TEXH4Pb removed the Help Wanted label Aug 25, 2025

AirBlack reviewed Aug 26, 2025

View reviewed changes

KIBORG04 reviewed Sep 5, 2025

View reviewed changes

github-actions bot added the Stalled PR label Oct 10, 2025

github-actions bot closed this Oct 17, 2025

TEXH4Pb reopened this Oct 18, 2025

github-actions bot removed the Stalled PR label Oct 18, 2025

github-actions bot added the Stalled PR label Nov 11, 2025

volas added the Pinned ПРы с этим лэйблом будут игнорироваться Stale-ботом label Nov 16, 2025

volas marked this pull request as draft November 16, 2025 03:04

Uh oh!

Comments

Conversation

TEXH4Pb commented Jul 22, 2025

Описание изменений

Почему и что этот ПР улучшит

Авторство

Чеинжлог

Uh oh!

TauKitty commented Jul 22, 2025

Changelog status: ✔️

Uh oh!

TEXH4Pb commented Jul 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

RichardJones1 commented Aug 10, 2025

Uh oh!

TEXH4Pb commented Aug 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

DarthSidiousPalpatine commented Aug 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

TEXH4Pb commented Aug 25, 2025

Uh oh!

TEXH4Pb commented Aug 25, 2025

Uh oh!

TEXH4Pb commented Aug 25, 2025

Uh oh!

TEXH4Pb commented Aug 25, 2025

Uh oh!

TEXH4Pb commented Aug 25, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

AirBlack commented Aug 26, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

KIBORG04 commented Sep 5, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

AirBlack commented Sep 6, 2025

Uh oh!

TEXH4Pb commented Sep 16, 2025

Uh oh!

github-actions bot commented Oct 10, 2025

Uh oh!

github-actions bot commented Oct 10, 2025

Uh oh!

github-actions bot commented Oct 17, 2025

Uh oh!

github-actions bot commented Nov 11, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

TEXH4Pb commented Jul 22, 2025 •

edited

Loading

TEXH4Pb commented Aug 17, 2025 •

edited

Loading

DarthSidiousPalpatine commented Aug 21, 2025 •

edited

Loading