Trailcode/hot keys

[trailcode]

Note

Medium Risk
Moderate risk because it adds new global hotkey handling and introduces platform-specific system() calls to open local documentation files, plus build changes that affect packaging/preloading of resources.

Overview
Adds Blender-style view roll: Shift+NumPad 4/6 now rotates the 3D view by a configurable step, persisted as gui.view_roll_step_deg (default 45°) and exposed in Settings → 3D view navigation.

Introduces offline help packaging: Markdown docs/licenses are copied (native) or preloaded (Emscripten) into res/doc/, and Help → Usage Guide (plus the new ? help next to the roll setting) opens the packaged file when available, otherwise falling back to GitHub.

Adds supporting infrastructure: Occt_view::roll_view_z_deg, an executable-directory helper in paths.*, updates default settings JSON and scripting help text, and includes new documentation (usage-occt-view.md) plus a couple of repo-local issue drafts and a small PDF-to-PNG conversion script.

^{Reviewed by Cursor Bugbot for commit 739cae9. Bugbot is set up for automated code reviews on this repo. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 2 potential issues.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 739cae9. Configure here.}

[cursor]

Assert crashes app on out-of-range user settings value

High Severity

EZY_ASSERT_MSG is used to validate the user-editable view_roll_step_deg setting, but this macro calls DEBUG_BREAK (__debugbreak / raise(SIGTRAP)) unconditionally in all builds (it's not guarded by NDEBUG). If a user manually sets view_roll_step_deg outside [0.1, 180] in their settings JSON, the app crashes on startup. Every other setting in this function uses silent range checks with fallback or std::clamp for invalid values — this is the only one that asserts on user-provided data.

 

^{Reviewed by Cursor Bugbot for commit 739cae9. Configure here.}

[cursor]

New function duplicates existing open_url_ logic

Low Severity

The new open_local_file_default_app_ function duplicates the platform-specific std::system command construction already present in open_url_. Both build the identical start / open / xdg-open command string. The caller open_packaged_doc_or_url_ already checks file existence before calling, making the existence check inside this helper also redundant. A call to open_url_(p.string().c_str()) would achieve the same result without duplicating the shell-command logic, reducing the risk of fixing a bug in one copy but not the other.

 

^{Reviewed by Cursor Bugbot for commit 739cae9. Configure here.}