Skip to content
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.

Sign up for GitHub

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

Jump to bottom

TGUI для Identification Computer #13008

Open
wants to merge 38 commits into
base: master
Choose a base branch
from
Open

TGUI для Identification Computer #13008

wants to merge 38 commits into from

Conversation

Slavik2001
Copy link
Contributor

@Slavik2001 Slavik2001 commented Mar 28, 2024 •
edited
Loading

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

Замена НаноУИ на ТГУИ
image

Пушнул, так как понял что это чуток сложнее и может затянутся на очень долго, а с поддержкой остальным прогресс будет быстрее

UPD: Всё готово

С useLocalState у меня не получилось, так что пока без него

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

Красиво и часть #7347

Авторство

Я+люди которые мне тут помогут, часть кода взята с https://github.com/ss220-space/Paradise

Чеинжлог

🆑 Slavik2001

  • image: TGUI для Identification Computer.

@TauKitty
Copy link
Contributor

Changelog status: ✔️

@TauKitty TauKitty added Sprites UI User Interface labels Mar 28, 2024
@AirBlack
Copy link
Contributor

Бери с ТГ лучше наверн, а так красава

AirBlack
AirBlack suggested changes Mar 29, 2024
View reviewed changes
code/game/machinery/computer/card.dm Show resolved Hide resolved
code/game/machinery/computer/card.dm Outdated Show resolved Hide resolved
@Slavik2001
Copy link
Contributor Author

Бери с ТГ лучше наверн, а так красава

Я бы взял, но когда я посмотрел на консоль у ТГ, то мне показалось что там намного сложнее
Screenshot_20240329-135251

@AirBlack
Copy link
Contributor

ок

@Slavik2001
Copy link
Contributor Author

Slavik2001 commented Mar 29, 2024 •
edited
Loading

Вооооот..... На текущем этапе вся вкладка Аксеса работает как надо, но я хз как сделать манифест и принт, пока что.....
Разве что денай регион снимает не всё сразу, а сначала те что бы добавлены регионом/профой и только потом те то были вручную....

@4310V343k
Copy link
Member

зачем тут две секции?
image

@Slavik2001
Copy link
Contributor Author

Slavik2001 commented Mar 29, 2024 •
edited
Loading

Внутри аксеса уже есть свой лейбл/тайтл, я просто не знал и когда добавлял, то вписал тайтл этому сегменту XD

@Slavik2001
Copy link
Contributor Author

Вотъ
image

@Slavik2001
Copy link
Contributor Author

В принципе... Всё готово и можно мержить, но я бы хотел увидить пока в ТМе и посмотреть что скажут насчет этого.

@Slavik2001 Slavik2001 changed the title [WiP] TGUI для Identification Computer TGUI для Identification Computer Mar 30, 2024
@volas
Copy link
Member

volas commented May 19, 2024

посмотри комментарии пикачушки, с mode очень актуально, иначе можно с клиента отправить не валидный и тем самым скорее всего сломать консоль

@volas volas removed the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label May 29, 2024
volas
volas reviewed Jun 8, 2024
View reviewed changes
code/game/machinery/computer/card.dm Outdated Show resolved Hide resolved
code/game/machinery/computer/card.dm Outdated Show resolved Hide resolved
@Slavik2001
Copy link
Contributor Author

Локал мод добавил, а что с списком должностей хз че делать

@volas volas added the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Jun 10, 2024
@volas
Copy link
Member

volas commented Jun 10, 2024

Можешь на бэке проверять существование по тем же command_positions и прочим, на фронте не обязательно. Если центкомовский - еще по цекашному списку

@simb11
Copy link
Contributor

simb11 commented Jun 15, 2024

при клике по select all менюшка ломается
image

@Slavik2001
Copy link
Contributor Author

Slavik2001 commented Jun 15, 2024 •
edited
Loading

Я уже писал, что при отсутствии навыка будет выдавать синий экран ТГУИ. То есть привязка к навыкам есть для выдачи вообще полного или полного для отдела.
На данный момент я хз как сделать чтобы этой кнопки не было если нету навыка.

Хочешь юзать эту кнопку, увеличь навык до 4
image

@volas volas removed the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Jun 18, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 2, 2024

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

@AzzyDreemurr13
Copy link
Contributor

@Slavik2001 не помирай

@Slavik2001
Copy link
Contributor Author

Slavik2001 commented Jul 4, 2024 •
edited
Loading

Теперь должно быть всё готово. Кнопки теперь исчезают и появляются при наличии/отсутствии навыков

@Slavik2001
Copy link
Contributor Author

Сделал. Проверяй

@volas volas added the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Jul 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maleyvich maleyvich maleyvich left review comments

@AirBlack AirBlack AirBlack requested changes

@NinjaPikachuska NinjaPikachuska Awaiting requested review from NinjaPikachuska

@volas volas Awaiting requested review from volas

Assignees
No one assigned
Labels
Sprites Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже UI User Interface
Projects
None yet
Milestone
NanoUI/BrowserUI --> TGUI
Development

Successfully merging this pull request may close these issues.

None yet
9 participants
@Slavik2001 @TauKitty @AirBlack @4310V343k @NinjaPikachuska @volas @AzzyDreemurr13 @simb11 @maleyvich