-
Notifications
You must be signed in to change notification settings - Fork 1
New features #25
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
base: master
Are you sure you want to change the base?
New features #25
Conversation
Основные тесты производились на сборке билдов под 1.21.6
|
Забыл добавить в описании фич. |
|
Исправил отображение индикатора пустых слотов. |
Добавлена дополнительная клавиша для отправки из буфера обмена, а не из настроек мода.
|
Добавлена дополнительная клавиша для отправки из буфера обмена, а не из настроек мода. |
|
Фига себе, не ожидал, что будут люди, помогающие проекту) |
bleudev
left a comment
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.
Пару изменений нужно сделать. Есть классные идеи, но хотелось бы сначала видеть issue, а затем пулл реквест) и делать всё очередно, ведь такие пуллы очень сложно контролировать. Сейчас мне нужно глянуть то, что уже было сделано (протестировать). Но я пока занят другой функцией + есть наконец идея для исправления извечного бага с рестарт баром. Поэтому когда исправлю буду работать вместе с тобой над этими функциями. Пока можешь исправить то, про что я написал. + могу сказать, что мне не нравится система папок, она слишком замудрённая (я про feature). Также, затем, нужно сделать обязательно порт всего этого на 1.21.9.
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.
Не вижу смысла и считаю прошлым веком реализовывать настройку через команды, только через конфиг.
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.
Не совсем уверен, что правильно понимаю. Реализация сделана таким образом, чтобы можно было управлять как через меню настроек мода, так и командами.
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.
Ну это лишние команды, я не думаю, что стоит их вообще добавлять
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.
Сколько людей, столько и мнений, но открывать каждый раз меню мода будет неудобно если есть в этом частая потребность. Учитывая, что большинство игроков не используют Modlist Memory или других модов запоминающих последний открыт мод через мод меню.
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.
можно сделать команду, которая откроет конфиг мода. это очень просто
1.21.6/src/main/java/com/bleudev/ppl_utils/mixin/client/ChatHudMixin.java
Outdated
Show resolved
Hide resolved
1.21.6/src/main/java/com/bleudev/ppl_utils/mixin/client/InventoryScreenMixin.java
Show resolved
Hide resolved
|
Короче я посмотрел. Фишки конечно крутые, но недоработанные. Я постараюсь их подправить (особенно мне не нравится счётчик алмазов). Идеи очень классные, но не понимаю зачем делать счётчик заполненных мест в инвентаре, это бесполезно. Скорее всего будет проще, если я открою новый пулл реквест с полным переписыванием фич и буду некоторые части кода копировать отсюда |
|
На будущее скажу, что лучше делать сначала issue, а затем уже, после обговорения функции, делать ее, а также делать по одной функции, желательно, ведь просто смотреть за этим сложно) |
|
Окей. Также присмотрись к изменением индикатора рестартов в Optimization and request, я оптимизировал его и добавил фишку со звуками. Насчёт утилиты с очередью команд, по большей части делалось под https://pplhub.info/holoart и https://pplhub.info/color Говоря про счётчик пустых слотов, это специфичная вещь которая нужна для сбора ресурсов, которые не выйдет фармить на фермах. К примеру, когда нужно добыть чего-то в большом объеме, майн устроен таким образом, что первым делом у тебя будет заполняться свободный хотбар, а уже после инвентарь. Если тебе нужно чего-то много и ты можешь добывать это быстро, но нужно собирать руками, придется каждый раз ориентироваться, а сколько еще могу собрать и че из мусора подобралось. Также хочу уточнить моменты. |
fixes #28
fixes #29
fixes #30
Новые фичи:
К примеру, вы используете ресурспак Пепеленда и устали каждый раз переименовывать золотую морковь в сырки или нужный вам тотем в ваш любимый скин. Теперь достаточно будет взять их в руку.
Использует команду /model имя на нужный тип предметов если игрок берет его в основную руку, предмет подходит по id и не имеет других значений по кастому.
Работает в формате
[minecraft:stick;"hand"], [minecraft:shield;"stop_shield"] - перечисление с помощью , отделения внутри ;
[id;"имя"]
Фильтрация чата
Проверка на совпадения с разными правилами фильтрации.
Два режима - blacklist(по умолчанию) и whitelist.
Blacklist - скрывает только при содержании совпадения.
Whitelist - отображает только при содержании совпадения.
Распространяется на ники и слова одновременно.
Счётчик Алмазов
Отображает количество алмазов и алмазных блоков в контейнере, также при наведении на любой предмет подсчитывает его общее количество.
К сожалению, не учитывает предметы внутри шалкеров.
Счётчик занятых слотов
Отображает текущее количество занятых слотов у игрока с учетом его хотбара и инвентаря.