misc mask fixes
authorCampbell Barton <ideasman42@gmail.com>
Wed, 1 Aug 2012 10:50:39 +0000 (10:50 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Wed, 1 Aug 2012 10:50:39 +0000 (10:50 +0000)
- image space used wrong notifiers.
- image notifier now checks for mask mode before listening to mask edits.
- mask keyframes now draw in the image space.

release/scripts/startup/bl_ui/space_image.py
source/blender/editors/include/ED_mask.h
source/blender/editors/mask/mask_draw.c
source/blender/editors/space_clip/clip_draw.c
source/blender/editors/space_clip/clip_intern.h
source/blender/editors/space_clip/clip_utils.c
source/blender/editors/space_image/space_image.c
source/blender/makesrna/intern/rna_mask.c
source/blender/makesrna/intern/rna_space.c

index e86c18aeba22b7f56abff130f4f07ed2cce64ab5..7752eb9771af520dbe2289b1a60460781237d1b0 100644 (file)
@@ -356,6 +356,7 @@ class IMAGE_HT_header(Header):
         show_render = sima.show_render
         # show_paint = sima.show_paint
         show_uvedit = sima.show_uvedit
+        show_maskedit = sima.show_maskedit
 
         row = layout.row(align=True)
         row.template_header()
@@ -407,7 +408,7 @@ class IMAGE_HT_header(Header):
             mesh = context.edit_object.data
             layout.prop_search(mesh.uv_textures, "active", mesh, "uv_textures", text="")
 
-        elif mode == 'MASK':
+        if show_maskedit:
             row = layout.row()
             row.template_ID(sima, "mask", new="mask.new")
 
index 9dd251149578d54fb2ddc71f1b1c6ab984d8dc94..88667729eee7ca4d8e9d5e971eb51e08f2ac467b 100644 (file)
@@ -61,6 +61,8 @@ void ED_mask_draw_region(struct Mask *mask, struct ARegion *ar,
                          float stabmat[4][4],
                          const bContext *C);
 
+void ED_mask_draw_frames(struct Mask *mask, struct ARegion *ar, const int cfra, const int sfra, const int efra);
+
 /* mask_shapekey.c */
 void ED_mask_layer_shape_auto_key(struct MaskLayer *masklay, const int frame);
 int ED_mask_layer_shape_auto_key_all(struct Mask *mask, const int frame);
index f01fad10034ec767c818181211f3158e4fc443f4..2f5a918a488746904a29445a43277fa4982f2671 100644 (file)
@@ -482,7 +482,7 @@ void ED_mask_draw_region(Mask *mask, ARegion *ar,
        struct View2D *v2d = &ar->v2d;
 
        int x, y;
-       int w, h;
+       /* int w, h; */
        float zoomx, zoomy;
 
        /* frame image */
@@ -492,8 +492,10 @@ void ED_mask_draw_region(Mask *mask, ARegion *ar,
        /* find window pixel coordinates of origin */
        UI_view2d_to_region_no_clip(&ar->v2d, 0.0f, 0.0f, &x, &y);
 
-       w = v2d->tot.xmax - v2d->tot.xmin;
-       h = v2d->tot.ymax - v2d->tot.ymin;
+
+       /* w = v2d->tot.xmax - v2d->tot.xmin; */
+       /* h = v2d->tot.ymax - v2d->tot.ymin;/*/
+
 
        zoomx = (float)(ar->winrct.xmax - ar->winrct.xmin + 1) / (float)((ar->v2d.cur.xmax - ar->v2d.cur.xmin));
        zoomy = (float)(ar->winrct.ymax - ar->winrct.ymin + 1) / (float)((ar->v2d.cur.ymax - ar->v2d.cur.ymin));
@@ -538,3 +540,32 @@ void ED_mask_draw_region(Mask *mask, ARegion *ar,
 
        glPopMatrix();
 }
+
+void ED_mask_draw_frames(Mask *mask, ARegion *ar, const int cfra, const int sfra, const int efra)
+{
+       const float framelen = ar->winx / (float)(efra - sfra + 1);
+
+       MaskLayer *masklay = BKE_mask_layer_active(mask);
+
+       glBegin(GL_LINES);
+       glColor4ub(255, 175, 0, 255);
+
+       if (masklay) {
+               MaskLayerShape *masklay_shape;
+
+               for (masklay_shape = masklay->splines_shapes.first;
+                    masklay_shape;
+                    masklay_shape = masklay_shape->next)
+               {
+                       int frame = masklay_shape->frame;
+
+                       /* draw_keyframe(i, CFRA, sfra, framelen, 1); */
+                       int height = (frame == cfra) ? 22 : 10;
+                       int x = (frame - sfra) * framelen;
+                       glVertex2i(x, 0);
+                       glVertex2i(x, height);
+               }
+       }
+
+       glEnd();
+}
index 96eb60028406e4235c8bd185004bce22b3c5ab38..c24cdab29e5466954d707b4ef357bb639f642539 100644 (file)
@@ -53,6 +53,7 @@
 
 #include "ED_screen.h"
 #include "ED_clip.h"
+#include "ED_mask.h"
 #include "ED_gpencil.h"
 
 #include "BIF_gl.h"
@@ -73,7 +74,7 @@
 
 /*********************** main area drawing *************************/
 
-void clip_draw_curfra_label(SpaceClip *sc, float x, float y)
+void clip_draw_curfra_label(const int framenr, const float x, const float y)
 {
        uiStyle *style = UI_GetStyle();
        int fontid = style->widget.uifont_id;
@@ -82,7 +83,7 @@ void clip_draw_curfra_label(SpaceClip *sc, float x, float y)
 
        /* frame number */
        BLF_size(fontid, 11.0f, U.dpi);
-       BLI_snprintf(numstr, sizeof(numstr), "%d", sc->user.framenr);
+       BLI_snprintf(numstr, sizeof(numstr), "%d", framenr);
 
        BLF_width_and_height(fontid, numstr, &font_dims[0], &font_dims[1]);
 
@@ -212,7 +213,7 @@ static void draw_movieclip_cache(SpaceClip *sc, ARegion *ar, MovieClip *clip, Sc
        UI_ThemeColor(TH_CFRAME);
        glRecti(x, 0, x + framelen, 8);
 
-       clip_draw_curfra_label(sc, x, 8.0f);
+       clip_draw_curfra_label(sc->user.framenr, x, 8.0f);
 
        /* solver keyframes */
        glColor4ub(175, 255, 0, 255);
@@ -221,24 +222,7 @@ static void draw_movieclip_cache(SpaceClip *sc, ARegion *ar, MovieClip *clip, Sc
 
        /* movie clip animation */
        if ((sc->mode == SC_MODE_MASKEDIT) && sc->mask_info.mask) {
-               MaskLayer *masklay = BKE_mask_layer_active(sc->mask_info.mask);
-               if (masklay) {
-                       MaskLayerShape *masklay_shape;
-
-                       glColor4ub(255, 175, 0, 255);
-                       glBegin(GL_LINES);
-
-                       for (masklay_shape = masklay->splines_shapes.first;
-                            masklay_shape;
-                            masklay_shape = masklay_shape->next)
-                       {
-                               i = masklay_shape->frame;
-
-                               draw_keyframe(i, CFRA, sfra, framelen, 1);
-                       }
-
-                       glEnd();
-               }
+               ED_mask_draw_frames(sc->mask_info.mask, ar, CFRA, sfra, efra);
        }
 }
 
index 3422aacf264f204d9f4daa80d6aaafc573b97a6e..392367f90718a9e3c2a82a64c0b16b0f81d6cd3e 100644 (file)
@@ -71,7 +71,7 @@ void CLIP_OT_dopesheet_select_channel(struct wmOperatorType *ot);
 /* clip_draw.c */
 void clip_draw_main(const struct bContext *C, struct SpaceClip *sc, struct ARegion *ar);
 void clip_draw_grease_pencil(struct bContext *C, int onlyv2d);
-void clip_draw_curfra_label(struct SpaceClip *sc, float x, float y);
+void clip_draw_curfra_label(const int framenr, const float x, const float y);
 
 /* clip_graph_draw.c */
 void clip_draw_graph(struct SpaceClip *sc, struct ARegion *ar, struct Scene *scene);
index 9e93aed6df7d62e8954df41bc51493451e9fd90c..ddc624b4cdfa3f80622d9408c741021efb2e9763 100644 (file)
@@ -263,7 +263,7 @@ void clip_draw_cfra(SpaceClip *sc, ARegion *ar, Scene *scene)
        UI_view2d_getscale(v2d, &xscale, &yscale);
        glScalef(1.0f / xscale, 1.0f, 1.0f);
 
-       clip_draw_curfra_label(sc, (float)sc->user.framenr * xscale, 18);
+       clip_draw_curfra_label(sc->user.framenr, (float)sc->user.framenr * xscale, 18);
 
        /* restore view transform */
        glScalef(xscale, 1.0, 1.0);
index 79cc1eba898fd4e408ec819f020fa0be78af8c69..f97813d3dd3d4fa6735faabf2d5e41ccb6a1e3b6 100644 (file)
@@ -430,22 +430,28 @@ static void image_listener(ScrArea *sa, wmNotifier *wmn)
                        }
                        break;
                case NC_MASK:
-                       switch (wmn->data) {
-                               case ND_SELECT:
-                               case ND_DATA:
-                               case ND_DRAW:
-                                       ED_area_tag_redraw(sa);
-                                       break;
-                       }
-                       switch (wmn->action) {
-                               case NA_SELECTED:
-                                       ED_area_tag_redraw(sa);
-                                       break;
-                               case NA_EDITED:
-                                       ED_area_tag_redraw(sa);
-                                       break;
+               {
+                       // Scene *scene = wmn->window->screen->scene;
+                       /* ideally would check for: ED_space_image_check_show_maskedit(scene, sima) but we cant get the scene */
+                       if (sima->mode == SI_MODE_MASK) {
+                               switch (wmn->data) {
+                                       case ND_SELECT:
+                                       case ND_DATA:
+                                       case ND_DRAW:
+                                               ED_area_tag_redraw(sa);
+                                               break;
+                               }
+                               switch (wmn->action) {
+                                       case NA_SELECTED:
+                                               ED_area_tag_redraw(sa);
+                                               break;
+                                       case NA_EDITED:
+                                               ED_area_tag_redraw(sa);
+                                               break;
+                               }
                        }
                        break;
+               }
                case NC_GEOM:
                        switch (wmn->data) {
                                case ND_DATA:
@@ -643,6 +649,8 @@ static void image_main_area_draw(const bContext *C, ARegion *ar)
                                    TRUE, FALSE,
                                    NULL, C);
 
+               ED_mask_draw_frames(mask, ar, CFRA, mask->sfra, mask->efra);
+
                draw_image_cursor(sima, ar);
        }
 
index 382f5501ca477c0065a0529337e74efe6fdc5bac..9ee2bb99b4e4c90b338c0b4b8574e589b16fdc79 100644 (file)
@@ -731,7 +731,7 @@ static void rna_def_mask(BlenderRNA *brna)
        RNA_def_property_int_funcs(prop, NULL, "rna_Mask_start_frame_set", NULL);
        RNA_def_property_range(prop, MINFRAME, MAXFRAME);
        RNA_def_property_ui_text(prop, "Start Frame", "First frame of the mask (used for sequencer)");
-       RNA_def_property_update(prop, NC_SCENE | ND_FRAME_RANGE, NULL);
+       RNA_def_property_update(prop, NC_MASK | ND_DRAW, NULL);
 
        prop = RNA_def_property(srna, "frame_end", PROP_INT, PROP_TIME);
        RNA_def_property_clear_flag(prop, PROP_ANIMATABLE);
@@ -739,7 +739,7 @@ static void rna_def_mask(BlenderRNA *brna)
        RNA_def_property_int_funcs(prop, NULL, "rna_Mask_end_frame_set", NULL);
        RNA_def_property_range(prop, MINFRAME, MAXFRAME);
        RNA_def_property_ui_text(prop, "End Frame", "Final frame of the mask (used for sequencer)");
-       RNA_def_property_update(prop, NC_SCENE | ND_FRAME_RANGE, NULL);
+       RNA_def_property_update(prop, NC_MASK | ND_DRAW, NULL);
 
        /* pointers */
        rna_def_animdata_common(srna);
index 6fd26e0f8c5c75346c2f08cc0b81771234864527..793eb2b7185d17ceee4763305824352e1c229ddd 100644 (file)
@@ -558,6 +558,13 @@ static int rna_SpaceImageEditor_show_uvedit_get(PointerRNA *ptr)
        return ED_space_image_show_uvedit(sima, sc->scene->obedit);
 }
 
+static int rna_SpaceImageEditor_show_maskedit_get(PointerRNA *ptr)
+{
+       SpaceImage *sima = (SpaceImage *)(ptr->data);
+       bScreen *sc = (bScreen *)ptr->id.data;
+       return ED_space_image_check_show_maskedit(sc->scene, sima);
+}
+
 static void rna_SpaceImageEditor_image_set(PointerRNA *ptr, PointerRNA value)
 {
        SpaceImage *sima = (SpaceImage *)(ptr->data);
@@ -1123,7 +1130,7 @@ void rna_def_space_mask_info(StructRNA *srna, int noteflag, const char *mask_set
        RNA_def_property_flag(prop, PROP_EDITABLE);
        RNA_def_property_ui_text(prop, "Mask", "Mask displayed and edited in this space");
        RNA_def_property_pointer_funcs(prop, NULL, mask_set_func, NULL, NULL);
-       RNA_def_property_update(prop, NC_SPACE | ND_SPACE_CLIP, NULL);
+       RNA_def_property_update(prop, noteflag, NULL);
 
        /* mask drawing */
        prop = RNA_def_property(srna, "mask_draw_type", PROP_ENUM, PROP_NONE);
@@ -2077,6 +2084,11 @@ static void rna_def_space_image(BlenderRNA *brna)
        RNA_def_property_clear_flag(prop, PROP_EDITABLE);
        RNA_def_property_ui_text(prop, "Show UV Editor", "Show UV editing related properties");
 
+       prop = RNA_def_property(srna, "show_maskedit", PROP_BOOLEAN, PROP_NONE);
+       RNA_def_property_boolean_funcs(prop, "rna_SpaceImageEditor_show_maskedit_get", NULL);
+       RNA_def_property_clear_flag(prop, PROP_EDITABLE);
+       RNA_def_property_ui_text(prop, "Show Mask Editor", "Show Mask editing related properties");
+
        rna_def_space_image_uv(brna);
 
        /* mask */