GPencil: Add Opacity y Onion switch to Dopesheet

Add new icons and panels Grease Pencil Dopesheet to manage layers without having the properties panel visible.

Also, the icons are in the same order in Dopesheet, Layers and Material list to keep consistency.

As the number of columns for icons is limited to 3 and we also need use a factor, I have impleted the change using slider area. Also, the slider option is enabled by default for 2D Template.

See T72026 for more info.

Reviewed By: mendio, pepeland, billreynish

Differential Revision: https://developer.blender.org/D6328

source/blender/blenloader/intern/versioning_defaults.c

@@ -218,6 +218,19 @@ static void blo_update_defaults_screen(bScreen *screen,
       }
     }
   }
+
+  /* 2D animation template. */
+  if (app_template && STREQ(app_template, "2D_Animation")) {
+    for (ScrArea *sa = screen->areabase.first; sa; sa = sa->next) {
+      for (ARegion *ar = sa->regionbase.first; ar; ar = ar->next) {
+        if (sa->spacetype == SPACE_ACTION) {
+          SpaceAction *saction = sa->spacedata.first;
+          /* Enable Sliders. */
+          saction->flag |= SACTION_SLIDERS;
+        }
+      }
+    }
+  }
 }
 
 void BLO_update_defaults_workspace(WorkSpace *workspace, const char *app_template)

source/blender/editors/animation/anim_channels_defines.c

@@ -4222,7 +4222,8 @@ void ANIM_channel_draw(
      * - Slider should start before the toggles (if they're visible)
      *   to keep a clean line down the side.
      */
-    if ((draw_sliders) && ELEM(ale->type, ANIMTYPE_FCURVE, ANIMTYPE_NLACURVE, ANIMTYPE_SHAPEKEY)) {
+    if ((draw_sliders) &&
+        ELEM(ale->type, ANIMTYPE_FCURVE, ANIMTYPE_NLACURVE, ANIMTYPE_SHAPEKEY, ANIMTYPE_GPLAYER)) {
       /* adjust offset */
       offset += SLIDER_WIDTH;
     }
@@ -4992,7 +4993,8 @@ void ANIM_channel_draw_widgets(const bContext *C,
      * - Slider should start before the toggles (if they're visible)
      *   to keep a clean line down the side.
      */
-    if ((draw_sliders) && ELEM(ale->type, ANIMTYPE_FCURVE, ANIMTYPE_NLACURVE, ANIMTYPE_SHAPEKEY)) {
+    if ((draw_sliders) &&
+        ELEM(ale->type, ANIMTYPE_FCURVE, ANIMTYPE_NLACURVE, ANIMTYPE_SHAPEKEY, ANIMTYPE_GPLAYER)) {
       /* adjust offset */
       /* TODO: make slider width dynamic,
        * so that they can be easier to use when the view is wide enough. */
@@ -5053,8 +5055,70 @@ void ANIM_channel_draw_widgets(const bContext *C,
           rna_path = BKE_keyblock_curval_rnapath_get(key, kb);
           free_path = 1;
         }
+        /* Special for Grease Pencil Layer. */
+        else if (ale->type == ANIMTYPE_GPLAYER) {
+          /* Add some offset to make it more pleasing to the eye. */
+          offset += SLIDER_WIDTH / 2.1f;
 
-        /* only if RNA-Path found */
+          char *gp_rna_path = NULL;
+          bGPDlayer *gpl = (bGPDlayer *)ale->data;
+          const short width = SLIDER_WIDTH / 5;
+
+          /* Create the RNA pointers. */
+          RNA_pointer_create(ale->id, &RNA_GPencilLayer, ale->data, &ptr);
+          RNA_id_pointer_create(ale->id, &id_ptr);
+          int icon;
+
+          /* Mask Layer. */
+          UI_block_emboss_set(block, UI_EMBOSS_NONE);
+          prop = RNA_struct_find_property(&ptr, "mask_layer");
+          gp_rna_path = RNA_path_from_ID_to_property(&ptr, prop);
+          if (RNA_path_resolve_property(&id_ptr, gp_rna_path, &ptr, &prop)) {
+            icon = (gpl->flag & GP_LAYER_USE_MASK) ? ICON_MOD_MASK : ICON_LAYER_ACTIVE;
+            uiDefAutoButR(
+                block, &ptr, prop, array_index, "", icon, offset, ymid, width, channel_height);
+          }
+          MEM_freeN(gp_rna_path);
+
+          /* Layer opacity. */
+          UI_block_emboss_set(block, UI_EMBOSS);
+          prop = RNA_struct_find_property(&ptr, "opacity");
+          gp_rna_path = RNA_path_from_ID_to_property(&ptr, prop);
+          if (RNA_path_resolve_property(&id_ptr, gp_rna_path, &ptr, &prop)) {
+            uiDefAutoButR(block,
+                          &ptr,
+                          prop,
+                          array_index,
+                          "",
+                          ICON_NONE,
+                          offset + width,
+                          ymid,
+                          width * 3,
+                          channel_height);
+          }
+          MEM_freeN(gp_rna_path);
+
+          /* Layer onion skinning switch. */
+          UI_block_emboss_set(block, UI_EMBOSS_NONE);
+          prop = RNA_struct_find_property(&ptr, "use_onion_skinning");
+          gp_rna_path = RNA_path_from_ID_to_property(&ptr, prop);
+          if (RNA_path_resolve_property(&id_ptr, gp_rna_path, &ptr, &prop)) {
+            icon = (gpl->onion_flag & GP_LAYER_ONIONSKIN) ? ICON_ONIONSKIN_ON : ICON_ONIONSKIN_OFF;
+            uiDefAutoButR(block,
+                          &ptr,
+                          prop,
+                          array_index,
+                          "",
+                          icon,
+                          offset + (width * 4),
+                          ymid,
+                          width,
+                          channel_height);
+          }
+          MEM_freeN(gp_rna_path);
+        }
+
+        /* Only if RNA-Path found. */
         if (rna_path) {
           /* get RNA pointer, and resolve the path */
           RNA_id_pointer_create(ale->id, &id_ptr);

source/blender/editors/gpencil/gpencil_utils.c

@@ -105,6 +105,7 @@ bGPdata **ED_gpencil_data_get_pointers_direct(
       /* XXX: Should we reduce reliance on context.gpencil_data for these cases? */
       case SPACE_PROPERTIES: /* properties */
       case SPACE_INFO:       /* header info (needed after workspaces merge) */
+      case SPACE_ACTION:     /* Dopesheet header. */
       {
         if (ob && (ob->type == OB_GPENCIL)) {
           /* GP Object */