-
Notifications
You must be signed in to change notification settings - Fork 13
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
Config Options #26
Comments
I'd be more than happy to implement the first two in a PR. for the second one, not sure where you'd want to do this as I'm not familiar with the codebase. |
I'm always open to quality of life improvements and you've put a lot of thought into this. I understand the point about the character limit in the input fields when you want to enter an equation. The background of this limitation was that it should not be possible to enter numbers that are not completely visible in the field. It would be possible to adapt the logic so that non-numeric expressions have no character limit and only the number itself has to be inside the box. On the subject of the GUI size, unfortunately, I cannot understand exactly what problem you are describing here. Which number exactly is 7 characters long and what is cut off? Can you prove your last assertion about the TPS problems with the requester using a profiler? That would surprise me because the requester has a stock change listener. It stores the known number of items in your network that you define in a requester slot and only when this number changes and is below the stock amount, an attempt is made to send another request. This means that new request operations are not started in every tick. Increasing the interval exponentially is therefore not a sensible approach here because the whole thing is event-based. Last but not least, because you didn't specify the version you are speaking about, I would like to mention that there will be no more changes to old versions. These are not critical bugs. Changes will therefore only be made for version 1.20.4+ for NeoForge. You are welcome to try implementing your wishes and I will take a look at the changes, but from the description, I can't really grasp what your goal is here. |
you could always check if the resulting number is greater than
I'll grab a screenshot of this later, when I get home. (if I don't forgor)
ah, that sucks that you can't change the width. oh well. edited the issue to remove that.
I can't prove it using a profiler, however, I have made the following observation:
also, I am on 1.20.1 fabric, so sadly it seems I won't get in updates. rip. |
Describe the feature/addition!
It would be nice if there were config options for the following things:
height/width of the requester interfacelinks to source:AbstractRequesterScreenNumberFieldStateBoxStatusDisplaySubmitButtoncurrently, if you use numbers that are 7 digits long, they can sometimes have 1 digit cut off (idfk why???). and if you increased the max digit count to >7, the extra digits would definitely be cut off.Why would like to see this feature/addition?
QOL
The text was updated successfully, but these errors were encountered: