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.

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

Redesign of the content addition menu #430

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

Xertis
Copy link
Contributor

@Xertis Xertis commented Jan 3, 2025

image

Ещё когда добавил поиск в меню контента, хотел добавить его ещё в меню добавления контента, но тогда нормальных идей, где разместить его не было, щас же она появилась.

Изменения

  1. Добавлен поиск (работает сразу на два поля)
  2. Добавлены стрелки для быстрого перемещения модов из двух полей
  3. Добавлены подсказки (крестик и галочка сверху) о предназначении полей
  4. Исправлен старый баг, при котором если выбрать генератор из стороннего мода, а после сам этот мод убрать из мира, вылезала ошибка в консоль, а мир не создавался, теперь же генератор будет сбрасываться до дефолтного (как изначально и было задумано)

function apply()
core.reconfig_packs(add_packs, rem_packs)
if mode ~= "world" then
menu:back()
end
end

function refresh_search()
local search_text = document.search_textbox.text:lower()
local interval = 2
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Позиционирование элементов в панелях происходит автоматически, так что нужно дождаться слияния ветки headless-mode, после чего добавлю корректный способ прятать элементы, вместо манипуляций с позициями.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ну тут скорее их надо не прятать, а сортировать

res/layouts/pages/content.xml.lua Show resolved Hide resolved
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