Maniphest T31830

UV Editing select by patch sometimes fails
Closed, ArchivedTO DO

Assigned To
None
Authored By
Reiner Prokein (tiles)
Jun 14 2012, 3:08 PM
Tags
  • BF Blender
Subscribers
Anthony Edlin (krash)
Reiner Prokein (tiles)
Sergey Sharybin (sergey)

Description

r47319 Win 7, 64 Bit

I noticed that selecting by patch in the UV editing fails sometimes. It sometimes selects a neighbour patch, or doesn`t select a patch at all. The attached scene has nearly every face as a single patch, i`m in the middle of unwrapping. I am not sure if this might even be the problem. But here it is extreme. Some patches are unselectable by that. I have to work around by using face mode.

Attached the example scene and a short flash video to show the problem

Event Timeline

Reiner Prokein (tiles) edited a custom field.Jun 14 2012, 3:08 PM
Reiner Prokein (tiles) attached 1 file(s): F20884: selectbypatchfails.swf.
Reiner Prokein (tiles) attached 1 file(s): F20885: selectbypatch.rar.
Anthony Edlin (krash) added a comment.Jun 17 2012, 11:09 PM

Hi Reiner,

Island selection I think works by first finding the closest vertex from where you clicked, then selecting that island. When I look at your video it seems to follow this convention. It can be confusing because you have the mouse over an island, but the closest vertex is part of a nearby island so it selects that island. If you go into face mode of UV editor on the scene you attached you can also see that it works similarly with faces, except that it first finds the closest middle of a face and then selects that face, which can also lead to times when your cursor is over a face but it selects some other nearby face whose center is closer then the face your cursor is over.

This works the same in the 3d view if you are viewing it in wireframe mode with face select, it selects faces by the closest middle of a face, not actually the face your cursor is over. In 3d view shaded mode it actually selects by which face your cursor is over. Either way, if you understand this is how it works in the UV editor you can select any island you need to.

It does seem unintuative though and could probably be fixed, but I'm not familiar with the code so would let someone else check this first. I think there is a GSoC student working on UV system this year and this might fit their responsibilities. If no one else responds for a couple of weeks I'll have a look at it when I'm not as busy.

Thanks for the report and the time making your video.

Reiner Prokein (tiles) added a comment.Jun 18 2012, 9:09 AM

I see. Thanks for the explanation. It is really confusing. But now i know at least where it comes from :)

Sergey Sharybin (sergey) added a comment.Jun 18 2012, 4:06 PM

This probably could be improved somehow, but wouldn't consider this is a bug.

Sergey Sharybin (sergey) changed the task status from Unknown Status to Unknown Status.Jun 18 2012, 4:06 PM
Reiner Prokein (tiles) added a comment.Jun 18 2012, 6:23 PM

Well. I do. I want to select patches, not vertices. But it`s of course your decision :)

Anthony Edlin (krash) added a comment.Jun 19 2012, 3:03 AM

Hi

After thinking about this a bit more and running through some actual workflows, it wouldn't make sense in uv mode to select the same way as shaded face selection in the 3d view, as sometimes in uv editor the uv's overlap on purpose, and there wouldn't be a good way to choose which should be on top.

That being said, I think it would make sense for island select mode to select by nearest vert, edge, or face, instead of just by nearest vert. This also sounds like a relatively easy patch so I wrote it up.

You can try it out here: http://projects.blender.org/tracker/index.php?func=detail&aid=31869&group_id=9&atid=127

Thanks.

Reiner Prokein (tiles) added a comment.Jun 19 2012, 9:02 AM

This sounds like a useful approach. Thanks :)

Anthony Edlin (krash) added a comment.Oct 29 2012, 9:46 PM

Hi,

I've updated the patch to work on trunk along with a couple of changes.

Also made a video demonstration of the patch here: https://www.youtube.com/watch?v=Vngz1JJwm1M