Skip to content

Ramming dialog improvements & add step control for SpinInput class #9651

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

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

Conversation

yw4z
Copy link
Contributor

@yw4z yw4z commented May 16, 2025

Tested on windows 11

CHANGES

• Added dark mode support for dialog
• Replaced wxSpinCtrl with SpinInput to match UI
• Added side texts to SpinInput's
• Used miliseconds on total ramming time to prevent usage of SpinInput with double values
• Added scaling support for buttons and tick lines
• Added cursor change while hovering and dragging buttons
• Softened colors to fix raw color palette. it looks a bit modern now
• Used orca color on buttons
• Used orange color on "NO RAMMING AT ALL" text because its warning
• Fixed alignment problems on graph values
• Used "0" on time value to match look with "0" value used on y axis (volumetric flow). previously "0.0" used on time that looks like it didnt aligned properly
• Fixed many minor issues
• updates dialog buttons

Spin Input changes
• Added an optional step control

COMPARISON

Before-After - Light theme
Screenshot-20250516180834
Screenshot-20250604084443

Before-After - Dark theme
Screenshot-20250516180937
Screenshot-20250604084353

Before-After - %200 scaling - Buttons and tick lines are very tiny
Screenshot-20250516175153
Screenshot-20250516175127

Preview of Cursor (PLAY TO PREVIEW)
orca-slicer_2X6t3aH2no

@MhzDev
Copy link
Contributor

MhzDev commented May 31, 2025

Nice work!
I know it's not the scope of the PR but I thought it would be useful to suggest :)
Would you consider expanding the ranges of the configuration to also work with prusa machines? (Not necessarily on this PR)
Basically mk4 and mmu3 requires a very high ramming value which is not possible on orca (the way I overcome this is by editing the config file manually without touching the UI).

More context: #9083
No pressure in any case, just an idea!

@yw4z
Copy link
Contributor Author

yw4z commented May 31, 2025

thanks, @SoftFever @Noisyfox check comment above about ramming on prusa machines

@Noisyfox
Copy link
Collaborator

Noisyfox commented Jun 4, 2025

Nice work! I know it's not the scope of the PR but I thought it would be useful to suggest :) Would you consider expanding the ranges of the configuration to also work with prusa machines? (Not necessarily on this PR) Basically mk4 and mmu3 requires a very high ramming value which is not possible on orca (the way I overcome this is by editing the config file manually without touching the UI).

More context: #9083 No pressure in any case, just an idea!

Two thoughs:

  1. Just simply increase the up limit to a larger number
  2. Automatically adjust the up limit based on current value. For example, the up limit is set to 1.5x the max value of current settings

Though I think this could be a separate PR.

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.

3 participants