mask/image viewer now works with non 1:1 image aspect, editing masks in the image...
authorCampbell Barton <ideasman42@gmail.com>
Wed, 25 Jul 2012 20:39:49 +0000 (20:39 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Wed, 25 Jul 2012 20:39:49 +0000 (20:39 +0000)
source/blender/blenkernel/BKE_mask.h
source/blender/blenkernel/intern/mask.c
source/blender/blenkernel/intern/movieclip.c
source/blender/editors/include/ED_image.h
source/blender/editors/mask/mask_edit.c
source/blender/editors/space_image/image_edit.c

index 9a4f30c853f6666eb3456a9ae235c773701eadab..98cab2e4062e78237962fa85b71974e65c095108 100644 (file)
@@ -127,7 +127,9 @@ void BKE_mask_free(struct Mask *mask);
 void BKE_mask_unlink(struct Main *bmain, struct Mask *mask);
 
 void BKE_mask_coord_from_movieclip(struct MovieClip *clip, struct MovieClipUser *user, float r_co[2], const float co[2]);
+void BKE_mask_coord_from_frame(float r_co[2], const float co[2], const float frame_size[2]);
 void BKE_mask_coord_to_movieclip(struct MovieClip *clip, struct MovieClipUser *user, float r_co[2], const float co[2]);
+void BKE_mask_coord_to_frame(float r_co[2], const float co[2], const float frame_size[2]);
 
 /* parenting */
 
index 6d49137d892b16a127155f33bbf9685e2cf69475..d963d46569a6f7fad97faee59777128eac5020cf 100644 (file)
@@ -1494,48 +1494,64 @@ void BKE_mask_unlink(Main *bmain, Mask *mask)
        mask->id.us = 0;
 }
 
-void BKE_mask_coord_from_movieclip(MovieClip *clip, MovieClipUser *user, float r_co[2], const float co[2])
+void BKE_mask_coord_from_frame(float r_co[2], const float co[2], const float frame_size[2])
 {
-       int width, height;
-
-       /* scaling for the clip */
-       BKE_movieclip_get_size(clip, user, &width, &height);
-
-       if (width == height) {
+       if (frame_size[0] == frame_size[1]) {
                r_co[0] = co[0];
                r_co[1] = co[1];
        }
-       else if (width < height) {
-               r_co[0] = ((co[0] - 0.5f) * ((float)width / (float)height)) + 0.5f;
+       else if (frame_size[0] < frame_size[1]) {
+               r_co[0] = ((co[0] - 0.5f) * (frame_size[0] / frame_size[1])) + 0.5f;
                r_co[1] = co[1];
        }
-       else { /* (width > height) */
+       else { /* (frame_size[0] > frame_size[1]) */
                r_co[0] = co[0];
-               r_co[1] = ((co[1] - 0.5f) * ((float)height / (float)width)) + 0.5f;
+               r_co[1] = ((co[1] - 0.5f) * (frame_size[1] / frame_size[0])) + 0.5f;
        }
 }
-
-/* as above but divide */
-void BKE_mask_coord_to_movieclip(MovieClip *clip, MovieClipUser *user, float r_co[2], const float co[2])
+void BKE_mask_coord_from_movieclip(MovieClip *clip, MovieClipUser *user, float r_co[2], const float co[2])
 {
        int width, height;
+       float frame_size[2];
 
        /* scaling for the clip */
        BKE_movieclip_get_size(clip, user, &width, &height);
 
-       if (width == height) {
+       frame_size[0] = (float)width;
+       frame_size[1] = (float)height;
+
+       BKE_mask_coord_from_frame(r_co, co, frame_size);
+}
+
+/* as above but divide */
+void BKE_mask_coord_to_frame(float r_co[2], const float co[2], const float frame_size[2])
+{
+       if (frame_size[0] == frame_size[1]) {
                r_co[0] = co[0];
                r_co[1] = co[1];
        }
-       else if (width < height) {
-               r_co[0] = ((co[0] - 0.5f) / ((float)width / (float)height)) + 0.5f;
+       else if (frame_size[0] < frame_size[1]) {
+               r_co[0] = ((co[0] - 0.5f) / (frame_size[0] / frame_size[1])) + 0.5f;
                r_co[1] = co[1];
        }
-       else { /* (width > height) */
+       else { /* (frame_size[0] > frame_size[1]) */
                r_co[0] = co[0];
-               r_co[1] = ((co[1] - 0.5f) / ((float)height / (float)width)) + 0.5f;
+               r_co[1] = ((co[1] - 0.5f) / (frame_size[1] / frame_size[0])) + 0.5f;
        }
 }
+void BKE_mask_coord_to_movieclip(MovieClip *clip, MovieClipUser *user, float r_co[2], const float co[2])
+{
+       int width, height;
+       float frame_size[2];
+
+       /* scaling for the clip */
+       BKE_movieclip_get_size(clip, user, &width, &height);
+
+       frame_size[0] = (float)width;
+       frame_size[1] = (float)height;
+
+       BKE_mask_coord_to_frame(r_co, co, frame_size);
+}
 
 static int BKE_mask_evaluate_parent(MaskParent *parent, float ctime, float r_co[2])
 {
index de367b6b4d0267efa3011ae6dbad9bd849649d28..045f0adaf41e13217638b7587a3f9641f6c77301 100644 (file)
@@ -1074,7 +1074,7 @@ void BKE_movieclip_reload(MovieClip *clip)
        else
                clip->source = MCLIP_SRC_SEQUENCE;
 
-       clip->lastsize[0] = clip->lastsize[1] = 0;
+       clip->lastsize[0] = clip->lastsize[1] = IMG_SIZE_FALLBACK;
        movieclip_load_get_szie(clip);
 
        movieclip_calc_length(clip);
index b31d52a6644da0474de3fe6d14caa7491b7bfcc5..393a400b7bdb2d460c19cf2543c1797a110d4cdf 100644 (file)
@@ -39,6 +39,7 @@ struct ToolSettings;
 struct uiBlock;
 struct wmWindowManager;
 struct ARegion;
+struct wmEvent;
 
 /* image_edit.c, exported for transform */
 struct Image *ED_space_image(struct SpaceImage *sima);
@@ -61,6 +62,7 @@ void ED_space_image_uv_sculpt_update(struct wmWindowManager *wm, struct ToolSett
 void ED_image_get_size(struct Image *ima, int *width, int *height);
 void ED_image_get_aspect(struct Image *ima, float *aspx, float *aspy);
 void ED_image_get_uv_aspect(struct Image *ima, float *aspx, float *aspy);
+void ED_image_mouse_pos(struct SpaceImage *sima, struct ARegion *ar, struct wmEvent *event, float co[2]);
 
 int ED_space_image_show_render(struct SpaceImage *sima);
 int ED_space_image_show_paint(struct SpaceImage *sima);
index 6667b9b69a58c043f7dedd5c903793e0fd6c6eeb..6a5e5c03345d3bcb36dec19858358c73d91359e3 100644 (file)
@@ -110,8 +110,15 @@ void ED_mask_mouse_pos(const bContext *C, wmEvent *event, float co[2])
                        }
                        case SPACE_IMAGE:
                        {
+                               int width, height;
+                               float frame_size[2];
+                               SpaceImage *sima = sa->spacedata.first;
                                ARegion *ar = CTX_wm_region(C);
-                               UI_view2d_region_to_view(&ar->v2d, event->mval[0], event->mval[1], &co[0], &co[1]);
+                               ED_space_image_get_size(sima, &width, &height);
+                               frame_size[0] = width;
+                               frame_size[1] = height;
+                               ED_image_mouse_pos(sima, ar, event, co);
+                               BKE_mask_coord_from_frame(co, co, frame_size);
                                break;
                        }
                        default:
index 8ae2b76cd4ee62edc582c1675814245c92e25a87..f80210fe4d7cba1edf4e81858dfe416ef7420a76 100644 (file)
@@ -47,6 +47,8 @@
 #include "ED_screen.h"
 #include "ED_uvedit.h"
 
+#include "UI_view2d.h"
+
 #include "WM_api.h"
 #include "WM_types.h"
 
@@ -250,6 +252,20 @@ void ED_image_get_uv_aspect(Image *ima, float *aspx, float *aspy)
        *aspy *= (float)h;
 }
 
+void ED_image_mouse_pos(SpaceImage *sima, ARegion *ar, wmEvent *event, float co[2])
+{
+       int sx, sy, width, height;
+       float zoomx, zoomy;
+
+       ED_space_image_get_zoom(sima, ar, &zoomx, &zoomy);
+       ED_space_image_get_size(sima, &width, &height);
+
+       UI_view2d_to_region_no_clip(&ar->v2d, 0.0f, 0.0f, &sx, &sy);
+
+       co[0] = ((event->mval[0] - sx) / zoomx) / width;
+       co[1] = ((event->mval[1] - sy) / zoomy) / height;
+}
+
 int ED_space_image_show_render(SpaceImage *sima)
 {
        return (sima->image && ELEM(sima->image->type, IMA_TYPE_R_RESULT, IMA_TYPE_COMPOSITE));