- User Since
- Apr 9 2020, 6:29 PM (12 w, 6 d)
Sun, Jul 5
Thu, Jun 25
Wed, Jun 24
Now here is just join and align algorithm
comments everywhere & bug fixes & simplifications
No more recursion. That is the whole join function. Will update patch soon
Tue, Jun 23
Join operator refactor. Small bug fixes
Mon, Jun 22
Now u can close any area
Sun, Jun 21
guess why he's showing only special cases. Anyway GOOD work
Sat, Jun 20
Tue, Jun 16
Mon, Jun 15
Sun, Jun 14
Ok, now u can join all areas. And all areas means any two adjacent areas.
That is all test function
Sat, Jun 13
Thu, Jun 11
Jun 9 2020
Updated patch its just implementation of test
That patch will not be affected by any changes of implementation functions like apply or getorientation (the second one is testing if areas can join ), that patch is not even about that, that's about how the user interacts with the operator.
So I think it's clear then that it should remain as a separate topic/patch.
And then it's ready for review.
If we are talking about ALL CSES when user can join two areas without the need to move edges, then your Patch is not doing that work, for now, maybe.
In the video below, u can see the difference between our implementations
Jun 8 2020
There is a refactor, for more info see comments bellow https://developer.blender.org/D7949#192723
And I fixed that (seems like it's true for now), though in a completely different way. Now I just need to make all looking good (I mean code).
If u will plan to work on that and didn't saw my cod, first let me know, please
No more "special edit types" at all. (Also clang-format was applied)
Jun 7 2020
Naming and bug fixes
Changed naming, added function "bool screen_geom_edge is movable" and few comments
Jun 6 2020
Now op can merge not obvious cases, thanks to the @Harley Acheson (harley) great idea
Jun 5 2020
"So screen_geom_find_area_whith_common_edge() could be altered to do similar to area_getorientation()..." - Yeah, I think it would be awesome. I definitely will try it later.