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

Improved Leveling Corner menu #2517

Merged
merged 5 commits into from
May 9, 2022

Conversation

digant73
Copy link
Contributor

@digant73 digant73 commented May 5, 2022

IMPROVEMENTS:

  • Improved Leveling Corner menu: Edge distance can be set in the range 0-100 (as in Manual Leveling menu). In case the edge distance is below max(probe offset X, probe offset Y) + 1 then a warning popup message is displayed asking to use the safe min value given by max(probe offset X, probe offset Y) + 1. The default edge distance (set when entering in the menu) is given by max(level_edge_distance, max(probe offset X, probe offset Y) + 1) where level_edge_distance is set in config.ini. Furthermore, when exiting from the menu the original edge distance value (for Manual Leveling menu) is restored.

resolves #2516

PR STATE: Ready for merge.

@bigtreetech bigtreetech merged commit d663c4c into bigtreetech:master May 9, 2022
kisslorand added a commit to kisslorand/BIGTREETECH-TFT that referenced this pull request May 12, 2022
kisslorand added a commit to kisslorand/BIGTREETECH-TFT that referenced this pull request May 21, 2022
mehmetsutas pushed a commit to mehmetsutas/BIGTREETECH-TouchScreenFirmware that referenced this pull request May 24, 2022
@radek8
Copy link
Contributor

radek8 commented Jun 12, 2022

@digant73, your edit is working, but the newly set value will not be saved.
If I leave the LevelCorner menu, the value will be overwritten by the original value.

@digant73
Copy link
Contributor Author

digant73 commented Jun 13, 2022

But the feature has nothing to save (even in the past). There was an issue that was fixed by PR #2525

@radek8
Copy link
Contributor

radek8 commented Jun 13, 2022

OK, it's working anyway. Thank you.
I made my own adjustment for myself.

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.

Auto bed leveling / L Corner controls configuration
3 participants