cleanup: style
[blender-staging.git] / source / blender / editors / space_clip / clip_buttons.c
index 18248f4775be8eeec1ede5ab1fc80ad815b3e610..c32d06cf9b175535ffd8ad0dc37a3c4af8a85cfd 100644 (file)
@@ -42,7 +42,6 @@
 #include "BLI_utildefines.h"
 #include "BLI_listbase.h"
 #include "BLI_path_util.h"
-#include "BLI_rect.h"
 #include "BLI_string.h"
 
 #include "BLF_translation.h"
@@ -53,7 +52,6 @@
 #include "BKE_movieclip.h"
 #include "BKE_tracking.h"
 
-#include "ED_clip.h"
 #include "ED_gpencil.h"
 
 #include "UI_interface.h"
 
 /* Panels */
 
-static int clip_grease_pencil_panel_poll(const bContext *C, PanelType *UNUSED(pt))
+void ED_clip_buttons_register(ARegionType *UNUSED(art))
 {
-       SpaceClip *sc = CTX_wm_space_clip(C);
 
-       return sc->view == SC_VIEW_CLIP;
-}
-
-void ED_clip_buttons_register(ARegionType *art)
-{
-       PanelType *pt;
-
-       pt = MEM_callocN(sizeof(PanelType), "spacetype clip panel gpencil");
-       strcpy(pt->idname, "CLIP_PT_gpencil");
-       strcpy(pt->label, N_("Grease Pencil"));
-       strcpy(pt->translation_context, BLF_I18NCONTEXT_DEFAULT_BPYRNA);
-       pt->draw_header = gpencil_panel_standard_header;
-       pt->draw = gpencil_panel_standard;
-       pt->flag |= PNL_DEFAULT_CLOSED;
-       pt->poll = clip_grease_pencil_panel_poll;
-       BLI_addtail(&art->paneltypes, pt);
 }
 
 /********************* MovieClip Template ************************/
@@ -130,18 +111,18 @@ void uiTemplateMovieClip(uiLayout *layout, bContext *C, PointerRNA *ptr, const c
        if (clip) {
                uiLayout *col;
 
-               row = uiLayoutRow(layout, FALSE);
+               row = uiLayoutRow(layout, false);
                block = uiLayoutGetBlock(row);
-               uiDefBut(block, LABEL, 0, IFACE_("File Path:"), 0, 19, 145, 19, NULL, 0, 0, 0, 0, "");
+               uiDefBut(block, UI_BTYPE_LABEL, 0, IFACE_("File Path:"), 0, 19, 145, 19, NULL, 0, 0, 0, 0, "");
 
-               row = uiLayoutRow(layout, FALSE);
-               split = uiLayoutSplit(row, 0.0f, FALSE);
-               row = uiLayoutRow(split, TRUE);
+               row = uiLayoutRow(layout, false);
+               split = uiLayoutSplit(row, 0.0f, false);
+               row = uiLayoutRow(split, true);
 
                uiItemR(row, &clipptr, "filepath", 0, "", ICON_NONE);
                uiItemO(row, "", ICON_FILE_REFRESH, "clip.reload");
 
-               col = uiLayoutColumn(layout, FALSE);
+               col = uiLayoutColumn(layout, false);
                uiTemplateColorspaceSettings(col, &clipptr, "colorspace_settings");
        }
 }
@@ -153,7 +134,7 @@ void uiTemplateTrack(uiLayout *layout, PointerRNA *ptr, const char *propname)
        PropertyRNA *prop;
        PointerRNA scopesptr;
        uiBlock *block;
-       rctf rect;
+       uiLayout *col;
        MovieClipScopes *scopes;
 
        if (!ptr->data)
@@ -175,16 +156,21 @@ void uiTemplateTrack(uiLayout *layout, PointerRNA *ptr, const char *propname)
        scopesptr = RNA_property_pointer_get(ptr, prop);
        scopes = (MovieClipScopes *)scopesptr.data;
 
-       rect.xmin = 0; rect.xmax = 10.0f * UI_UNIT_X;
-       rect.ymin = 0; rect.ymax = 6.0f * UI_UNIT_Y;
+       if (scopes->track_preview_height < UI_UNIT_Y) {
+               scopes->track_preview_height = UI_UNIT_Y;
+       }
+       else if (scopes->track_preview_height > UI_UNIT_Y * 20) {
+               scopes->track_preview_height = UI_UNIT_Y * 20;
+       }
 
-       block = uiLayoutAbsoluteBlock(layout);
+       col = uiLayoutColumn(layout, true);
+       block = uiLayoutGetBlock(col);
 
-       scopes->track_preview_height =
-               (scopes->track_preview_height <= 20) ? 20 : scopes->track_preview_height;
+       uiDefBut(block, UI_BTYPE_TRACK_PREVIEW, 0, "", 0, 0, UI_UNIT_X * 10, scopes->track_preview_height, scopes, 0, 0, 0, 0, "");
 
-       uiDefBut(block, TRACKPREVIEW, 0, "", rect.xmin, rect.ymin, BLI_rctf_size_x(&rect),
-                scopes->track_preview_height * UI_DPI_FAC, scopes, 0, 0, 0, 0, "");
+       /* Resize grip. */
+       uiDefIconButI(block, UI_BTYPE_GRIP, 0, ICON_GRIP, 0, 0, UI_UNIT_X * 10, (short)(UI_UNIT_Y * 0.8f),
+                     &scopes->track_preview_height, UI_UNIT_Y, UI_UNIT_Y * 20.0f, 0.0f, 0.0f, "");
 }
 
 /********************* Marker Template ************************/
@@ -238,7 +224,8 @@ static void marker_block_handler(bContext *C, void *arg_cb, int event)
 {
        MarkerUpdateCb *cb = (MarkerUpdateCb *) arg_cb;
        MovieTrackingMarker *marker;
-       int width, height, ok = FALSE;
+       int width, height;
+       bool ok = false;
 
        BKE_movieclip_get_size(cb->clip, cb->user, &width, &height);
 
@@ -252,7 +239,7 @@ static void marker_block_handler(bContext *C, void *arg_cb, int event)
                DAG_id_tag_update(&cb->clip->id, 0);
                WM_event_add_notifier(C, NC_SPACE | ND_SPACE_VIEW3D, NULL);
 
-               ok = TRUE;
+               ok = true;
        }
        else if (event == B_MARKER_PAT_DIM) {
                float dim[2], pat_dim[2], pat_min[2], pat_max[2];
@@ -276,7 +263,7 @@ static void marker_block_handler(bContext *C, void *arg_cb, int event)
 
                BKE_tracking_marker_clamp(cb->marker, CLAMP_PAT_DIM);
 
-               ok = TRUE;
+               ok = true;
        }
        else if (event == B_MARKER_SEARCH_POS) {
                float delta[2], side[2];
@@ -292,7 +279,7 @@ static void marker_block_handler(bContext *C, void *arg_cb, int event)
 
                BKE_tracking_marker_clamp(cb->marker, CLAMP_SEARCH_POS);
 
-               ok = TRUE;
+               ok = true;
        }
        else if (event == B_MARKER_SEARCH_DIM) {
                float dim[2], search_dim[2];
@@ -313,12 +300,12 @@ static void marker_block_handler(bContext *C, void *arg_cb, int event)
 
                BKE_tracking_marker_clamp(cb->marker, CLAMP_SEARCH_DIM);
 
-               ok = TRUE;
+               ok = true;
        }
        else if (event == B_MARKER_FLAG) {
                marker->flag = cb->marker_flag;
 
-               ok = TRUE;
+               ok = true;
        }
        else if (event == B_MARKER_OFFSET) {
                float offset[2], delta[2];
@@ -337,7 +324,7 @@ static void marker_block_handler(bContext *C, void *arg_cb, int event)
                DAG_id_tag_update(&cb->clip->id, 0);
                WM_event_add_notifier(C, NC_SPACE | ND_SPACE_VIEW3D, NULL);
 
-               ok = TRUE;
+               ok = true;
        }
 
        if (ok)
@@ -399,9 +386,9 @@ void uiTemplateMarker(uiLayout *layout, PointerRNA *ptr, const char *propname, P
                else
                        tip = TIP_("Marker is enabled at current frame");
 
-               bt = uiDefIconButBitI(block, TOGN, MARKER_DISABLED, 0, ICON_RESTRICT_VIEW_OFF, 0, 0, UI_UNIT_X, UI_UNIT_Y,
+               bt = uiDefIconButBitI(block, UI_BTYPE_TOGGLE_N, MARKER_DISABLED, 0, ICON_RESTRICT_VIEW_OFF, 0, 0, UI_UNIT_X, UI_UNIT_Y,
                                      &cb->marker_flag, 0, 0, 1, 0, tip);
-               uiButSetNFunc(bt, marker_update_cb, cb, NULL);
+               UI_but_funcN_set(bt, marker_update_cb, cb, NULL);
        }
        else {
                int width, height, step, digits;
@@ -411,9 +398,9 @@ void uiTemplateMarker(uiLayout *layout, PointerRNA *ptr, const char *propname, P
                BKE_movieclip_get_size(clip, user, &width, &height);
 
                if (track->flag & TRACK_LOCKED) {
-                       uiLayoutSetActive(layout, FALSE);
+                       uiLayoutSetActive(layout, false);
                        block = uiLayoutAbsoluteBlock(layout);
-                       uiDefBut(block, LABEL, 0, IFACE_("Track is locked"), 0, 0, UI_UNIT_X * 15.0f, UI_UNIT_Y, NULL, 0, 0, 0, 0, "");
+                       uiDefBut(block, UI_BTYPE_LABEL, 0, IFACE_("Track is locked"), 0, 0, UI_UNIT_X * 15.0f, UI_UNIT_Y, NULL, 0, 0, 0, 0, "");
 
                        return;
                }
@@ -438,53 +425,53 @@ void uiTemplateMarker(uiLayout *layout, PointerRNA *ptr, const char *propname, P
                cb->marker_flag = marker->flag;
 
                block = uiLayoutAbsoluteBlock(layout);
-               uiBlockSetHandleFunc(block, marker_block_handler, cb);
-               uiBlockSetNFunc(block, marker_update_cb, cb, NULL);
+               UI_block_func_handle_set(block, marker_block_handler, cb);
+               UI_block_funcN_set(block, marker_update_cb, cb, NULL);
 
                if (cb->marker_flag & MARKER_DISABLED)
                        tip = TIP_("Marker is disabled at current frame");
                else
                        tip = TIP_("Marker is enabled at current frame");
 
-               uiDefButBitI(block, OPTIONN, MARKER_DISABLED, B_MARKER_FLAG, IFACE_("Enabled"), 0.5 * UI_UNIT_X, 9.5 * UI_UNIT_Y, 7.25 * UI_UNIT_X, UI_UNIT_Y,
+               uiDefButBitI(block, UI_BTYPE_CHECKBOX_N, MARKER_DISABLED, B_MARKER_FLAG, IFACE_("Enabled"), 0.5 * UI_UNIT_X, 9.5 * UI_UNIT_Y, 7.25 * UI_UNIT_X, UI_UNIT_Y,
                             &cb->marker_flag, 0, 0, 0, 0, tip);
 
-               col = uiLayoutColumn(layout, TRUE);
+               col = uiLayoutColumn(layout, true);
                uiLayoutSetActive(col, (cb->marker_flag & MARKER_DISABLED) == 0);
 
                block = uiLayoutAbsoluteBlock(col);
-               uiBlockBeginAlign(block);
+               UI_block_align_begin(block);
 
-               uiDefBut(block, LABEL, 0, IFACE_("Position:"), 0, 10 * UI_UNIT_Y, 15 * UI_UNIT_X, UI_UNIT_Y, NULL, 0, 0, 0, 0, "");
-               uiDefButF(block, NUM, B_MARKER_POS, IFACE_("X:"), 0.5 * UI_UNIT_X, 9 * UI_UNIT_Y, 7.25 * UI_UNIT_X, UI_UNIT_Y, &cb->marker_pos[0],
+               uiDefBut(block, UI_BTYPE_LABEL, 0, IFACE_("Position:"), 0, 10 * UI_UNIT_Y, 15 * UI_UNIT_X, UI_UNIT_Y, NULL, 0, 0, 0, 0, "");
+               uiDefButF(block, UI_BTYPE_NUM, B_MARKER_POS, IFACE_("X:"), 0.5 * UI_UNIT_X, 9 * UI_UNIT_Y, 7.25 * UI_UNIT_X, UI_UNIT_Y, &cb->marker_pos[0],
                          -10 * width, 10.0 * width, step, digits, TIP_("X-position of marker at frame in screen coordinates"));
-               uiDefButF(block, NUM, B_MARKER_POS, IFACE_("Y:"), 8.25 * UI_UNIT_X, 9 * UI_UNIT_Y, 7.25 * UI_UNIT_X, UI_UNIT_Y, &cb->marker_pos[1],
+               uiDefButF(block, UI_BTYPE_NUM, B_MARKER_POS, IFACE_("Y:"), 8.25 * UI_UNIT_X, 9 * UI_UNIT_Y, 7.25 * UI_UNIT_X, UI_UNIT_Y, &cb->marker_pos[1],
                          -10 * height, 10.0 * height, step, digits,
                          TIP_("Y-position of marker at frame in screen coordinates"));
 
-               uiDefBut(block, LABEL, 0, IFACE_("Offset:"), 0, 8 * UI_UNIT_Y, 15 * UI_UNIT_X, UI_UNIT_Y, NULL, 0, 0, 0, 0, "");
-               uiDefButF(block, NUM, B_MARKER_OFFSET, IFACE_("X:"), 0.5 * UI_UNIT_X, 7 * UI_UNIT_Y, 7.25 * UI_UNIT_X, UI_UNIT_Y, &cb->track_offset[0],
+               uiDefBut(block, UI_BTYPE_LABEL, 0, IFACE_("Offset:"), 0, 8 * UI_UNIT_Y, 15 * UI_UNIT_X, UI_UNIT_Y, NULL, 0, 0, 0, 0, "");
+               uiDefButF(block, UI_BTYPE_NUM, B_MARKER_OFFSET, IFACE_("X:"), 0.5 * UI_UNIT_X, 7 * UI_UNIT_Y, 7.25 * UI_UNIT_X, UI_UNIT_Y, &cb->track_offset[0],
                          -10 * width, 10.0 * width, step, digits, TIP_("X-offset to parenting point"));
-               uiDefButF(block, NUM, B_MARKER_OFFSET, IFACE_("Y:"), 8.25 * UI_UNIT_X, 7 * UI_UNIT_Y, 7.25 * UI_UNIT_X, UI_UNIT_Y, &cb->track_offset[1],
+               uiDefButF(block, UI_BTYPE_NUM, B_MARKER_OFFSET, IFACE_("Y:"), 8.25 * UI_UNIT_X, 7 * UI_UNIT_Y, 7.25 * UI_UNIT_X, UI_UNIT_Y, &cb->track_offset[1],
                          -10 * height, 10.0 * height, step, digits, TIP_("Y-offset to parenting point"));
 
-               uiDefBut(block, LABEL, 0, IFACE_("Pattern Area:"), 0, 6 * UI_UNIT_Y, 15 * UI_UNIT_X, UI_UNIT_Y, NULL, 0, 0, 0, 0, "");
-               uiDefButF(block, NUM, B_MARKER_PAT_DIM, IFACE_("Width:"), 0.5 * UI_UNIT_X, 5 * UI_UNIT_Y, 15 * UI_UNIT_X, UI_UNIT_Y, &cb->marker_pat[0], 3.0f,
+               uiDefBut(block, UI_BTYPE_LABEL, 0, IFACE_("Pattern Area:"), 0, 6 * UI_UNIT_Y, 15 * UI_UNIT_X, UI_UNIT_Y, NULL, 0, 0, 0, 0, "");
+               uiDefButF(block, UI_BTYPE_NUM, B_MARKER_PAT_DIM, IFACE_("Width:"), 0.5 * UI_UNIT_X, 5 * UI_UNIT_Y, 15 * UI_UNIT_X, UI_UNIT_Y, &cb->marker_pat[0], 3.0f,
                          10.0 * width, step, digits, TIP_("Width of marker's pattern in screen coordinates"));
-               uiDefButF(block, NUM, B_MARKER_PAT_DIM, IFACE_("Height:"), 0.5 * UI_UNIT_X, 4 * UI_UNIT_Y, 15 * UI_UNIT_X, UI_UNIT_Y, &cb->marker_pat[1], 3.0f,
+               uiDefButF(block, UI_BTYPE_NUM, B_MARKER_PAT_DIM, IFACE_("Height:"), 0.5 * UI_UNIT_X, 4 * UI_UNIT_Y, 15 * UI_UNIT_X, UI_UNIT_Y, &cb->marker_pat[1], 3.0f,
                          10.0 * height, step, digits, TIP_("Height of marker's pattern in screen coordinates"));
 
-               uiDefBut(block, LABEL, 0, IFACE_("Search Area:"), 0, 3 * UI_UNIT_Y, 15 * UI_UNIT_X, UI_UNIT_Y, NULL, 0, 0, 0, 0, "");
-               uiDefButF(block, NUM, B_MARKER_SEARCH_POS, IFACE_("X:"), 0.5 * UI_UNIT_X, 2 * UI_UNIT_Y, 7.25 * UI_UNIT_X, UI_UNIT_Y, &cb->marker_search_pos[0],
+               uiDefBut(block, UI_BTYPE_LABEL, 0, IFACE_("Search Area:"), 0, 3 * UI_UNIT_Y, 15 * UI_UNIT_X, UI_UNIT_Y, NULL, 0, 0, 0, 0, "");
+               uiDefButF(block, UI_BTYPE_NUM, B_MARKER_SEARCH_POS, IFACE_("X:"), 0.5 * UI_UNIT_X, 2 * UI_UNIT_Y, 7.25 * UI_UNIT_X, UI_UNIT_Y, &cb->marker_search_pos[0],
                          -width, width, step, digits, TIP_("X-position of search at frame relative to marker's position"));
-               uiDefButF(block, NUM, B_MARKER_SEARCH_POS, IFACE_("Y:"), 8.25 * UI_UNIT_X, 2 * UI_UNIT_Y, 7.25 * UI_UNIT_X, UI_UNIT_Y, &cb->marker_search_pos[1],
+               uiDefButF(block, UI_BTYPE_NUM, B_MARKER_SEARCH_POS, IFACE_("Y:"), 8.25 * UI_UNIT_X, 2 * UI_UNIT_Y, 7.25 * UI_UNIT_X, UI_UNIT_Y, &cb->marker_search_pos[1],
                          -height, height, step, digits, TIP_("Y-position of search at frame relative to marker's position"));
-               uiDefButF(block, NUM, B_MARKER_SEARCH_DIM, IFACE_("Width:"), 0.5 * UI_UNIT_X, 1 * UI_UNIT_Y, 15 * UI_UNIT_X, UI_UNIT_Y, &cb->marker_search[0], 3.0f,
+               uiDefButF(block, UI_BTYPE_NUM, B_MARKER_SEARCH_DIM, IFACE_("Width:"), 0.5 * UI_UNIT_X, 1 * UI_UNIT_Y, 15 * UI_UNIT_X, UI_UNIT_Y, &cb->marker_search[0], 3.0f,
                          10.0 * width, step, digits, TIP_("Width of marker's search in screen coordinates"));
-               uiDefButF(block, NUM, B_MARKER_SEARCH_DIM, IFACE_("Height:"), 0.5 * UI_UNIT_X, 0 * UI_UNIT_Y, 15 * UI_UNIT_X, UI_UNIT_Y, &cb->marker_search[1], 3.0f,
+               uiDefButF(block, UI_BTYPE_NUM, B_MARKER_SEARCH_DIM, IFACE_("Height:"), 0.5 * UI_UNIT_X, 0 * UI_UNIT_Y, 15 * UI_UNIT_X, UI_UNIT_Y, &cb->marker_search[1], 3.0f,
                          10.0 * height, step, digits, TIP_("Height of marker's search in screen coordinates"));
 
-               uiBlockEndAlign(block);
+               UI_block_align_end(block);
        }
 }
 
@@ -522,7 +509,7 @@ void uiTemplateMovieclipInformation(uiLayout *layout, PointerRNA *ptr, const cha
        clip = (MovieClip *)clipptr.data;
        user = userptr->data;
 
-       col = uiLayoutColumn(layout, FALSE);
+       col = uiLayoutColumn(layout, false);
 
        ibuf = BKE_movieclip_get_ibuf_flag(clip, user, clip->flag, MOVIECLIP_CACHE_SKIP);
 
@@ -553,7 +540,7 @@ void uiTemplateMovieclipInformation(uiLayout *layout, PointerRNA *ptr, const cha
        uiItemL(col, str, ICON_NONE);
 
        /* Display current frame number. */
-       framenr = BKE_movieclip_remap_scene_to_clip_frame(clip, user->framenr) ;
+       framenr = BKE_movieclip_remap_scene_to_clip_frame(clip, user->framenr);
        if (framenr <= clip->len)
                BLI_snprintf(str, sizeof(str), IFACE_("Frame: %d / %d"), framenr, clip->len);
        else