Skip to content

фикс иконок в group на кнопки тогглы и т.д. - #110

Merged
Footagesus merged 2 commits into
Footagesus:mainfrom
itzdanti:main
Oct 7, 2026
Merged

Footagesus merged 2 commits into
Footagesus:mainfrom
itzdanti:main

Conversation

@itzdanti

@itzdanti itzdanti commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

привет фута, у тебя короче код хрень тота сёта, кратко:

  • creator.icon теперь никогда не возвращает nil. если иконку не нашли - возвращается безопасный fallback в том же формате: пустая строка для картинки и нулевые vector2 для rect position/size. валидные иконки возвращаются ровно как раньше, для них ничего не поменялось
  • заодно захардил все места, где результат creator.icon индексировали без проверки (toggle, button, input, label, code, dropdown, section, notification, поиск, window init, openbutton) - теперь вызов выносится в локальную переменную и проверяется на nil перед индексацией
  • в creator.image проверку "это вообще резолвится как иконка?" перевел на icons.icon2 напрямую, чтобы url-картинки и fallback-ветка работали как раньше

сборка кстати была сломана, она не читала сабмодули (icons - сабмодуль)

image

@itzdanti

itzdanti commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

@Footagesus я вижу лайка и не вижу добавления этого в репо, у тебя нахуй сломаны кнопки тогглы и т.д. в группах

@Footagesus

Copy link
Copy Markdown
Owner

я ща чуть занят

@itzdanti

itzdanti commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

я ща чуть занят

ланаа

@itzdanti

itzdanti commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

ой бля

@itzdanti

itzdanti commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

сломалось

@Footagesus
Footagesus merged commit 4851d03 into Footagesus:main Oct 7, 2026
0 of 2 checks passed
@itzdanti

itzdanti commented Oct 7, 2026

Copy link
Copy Markdown
Contributor Author

ЕЕЕ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants