Skip to content

Conversation

@Micheus
Copy link
Collaborator

@Micheus Micheus commented Oct 22, 2025

This can be particularly useful for text-based texturing.

Note: The RMB option helps preserve the layout of UV islands as displayed in the viewer. Thanks to envelupo for the suggestion.

This can be particularly useful for text-based texturing.

Note: The RMB option helps preserve the layout of UV islands as displayed in
the viewer. Thanks to envelupo for the suggestion.
@Micheus
Copy link
Collaborator Author

Micheus commented Oct 22, 2025

The default LMB option was named in the code as 'arrange' mode:
uv-view projection-arrange

and the RMB option is 'preserve':
uv-view projection-preserve

@dgud dgud requested a review from Copilot December 2, 2025 14:43
@dgud
Copy link
Owner

dgud commented Dec 2, 2025

I have let co-pilot review this, fix stuff that seems valid and write when your are done, or nothing needs fixing.
It is wrong often.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a right mouse button (RMB) option to the Projection Camera feature in the UV Segmentation window, allowing users to choose between arranging UV islands (default behavior) or preserving their projection layout as seen in the viewer.

Key Changes:

  • Modified the camera projection type from an atom to a tuple {camera, Mode} to support two modes: arrange and preserve
  • Added UI menu option with RMB functionality for mode selection
  • Implemented new place_areas/2 function with preserve mode logic

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
wpc_autouv.erl Added new 5-arity init_show_maps function with UVMode parameter; exported both 4 and 5-arity versions
auv_util.erl Added uv_mode/1 helper function to extract mode from camera tuple; updated exports
auv_seg_ui.erl Added camera_mode/0 function for RMB menu handling; updated pattern matching for camera tuple; integrated mode selection in chart mapping workflow
auv_placement.erl Implemented new place_areas/2 with preserve mode; added bbox/1 helper; defined HUGE/TINY constants for bounding box calculations
auv_mapping.erl Updated pattern matching to handle {camera, _} tuple instead of camera atom

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants