Skip to content

#14362 Allow arbitrary Z-scale factor - #14412

Open
magnesj wants to merge 5 commits into
OPM:devfrom
magnesj:arbitrary-z-scale-14362
Open

#14362 Allow arbitrary Z-scale factor#14412
magnesj wants to merge 5 commits into
OPM:devfrom
magnesj:arbitrary-z-scale-14362

Conversation

@magnesj

@magnesj magnesj commented Jul 30, 2026

Copy link
Copy Markdown
Member

Closes #14362

Summary

  • Make the Z-scale combo boxes in the main window toolbar and the view property editor editable, so any positive scale factor can be entered as free text. Input is validated: non-numeric or non-positive values are rejected and the previous scale is restored.
  • The current scale value is always included in the property editor option list, so arbitrary values loaded from a project or entered in the toolbar display correctly.
  • Add keyboard shortcuts Ctrl+Shift+Up / Ctrl+Shift+Down (also available in the View menu as Increase Z Scale / Decrease Z Scale, with new icons) to step the Z-scale of the active view through the predefined scale values.
  • Make Rim3dView::isScaleZEditable virtual and const, overridden in RimEclipseContourMapView to return false, replacing dynamic_cast checks.

magnesj added 5 commits July 30, 2026 10:07
Make the Z-scale combo boxes in the toolbar and property editor editable, so any positive scale value can be entered as text. Invalid input is rejected and the previous value is restored.

Add keyboard shortcuts Ctrl+Shift+Up and Ctrl+Shift+Down to step the Z-scale of the active view through the predefined scale values.
RimEclipseView::defineEditorAttribute did not call the base class implementation, so the editable Z-scale combo box attribute defined in Rim3dView was never applied for Eclipse views.
Make Rim3dView::isScaleZEditable virtual and const, and return false for contour map views. Replace the dynamic_cast checks on RimEclipseContourMapView in RiuMainWindow and RicZScaleFeatures with the virtual function. Make viewsUsingThisAsComparisonView const to support this.
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.

Allow arbitrary Z-scale factor

1 participant