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

Disable reclaim and resource sliders if the respective generator is set to random #3222

Open
BlackYps opened this issue Jul 16, 2024 · 2 comments

Comments

@BlackYps
Copy link
Collaborator

Is your feature request related to a problem?

People use the reclaim sliders (for example setting it to low) while keeping the prop generator at random and are then frustrated when a high reclaim map comes out because a prop generator got selected that always produces a lot of reclaim.

Describe the solution you'd like

We should disable the sliders when the respective generator is set to random.

Describe alternatives you've considered

ALternatively having the slider restrict which generators can be chosen is the behaviour that we just went away from and it is too opaque for the user.

@sulikdan
Copy link
Contributor

@BlackYps Would you have any more description about relations between buttons/sliders? I could try to disable some of them and also add hover-hint message ...

Do I understand correctly:
Resource Generation set Random -> disable Resource Density slider
Prop Generation set Random -> disable Reclaim Density slider

?

What about Symmetry? There is also random option. Are there any relations? E.g. Fixed seed or even map style?

Anything else?

@BlackYps
Copy link
Collaborator Author

yeah, these two that you describe are correct. The symmetry has no implications for the other settings. I don't think we need any other. Some other disabling logic is already implemented and these two seem to be the only ones missing.

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

No branches or pull requests

2 participants