feature: added frames

7 files changed, 247 insertions, 91 deletions

diff --git a/src/action.h b/src/action.h
index 67f9077..a1955a0 100644
--- a/src/action.h
+++ b/src/action.h
@@ -12,7 +12,7 @@ struct action_pixcol {
 
 struct ActionPixelsColors {
 	long int cnt;
-	unsigned lyr;
+	unsigned layer, frame;
 	struct action_pixcol *pix;
 };
 
diff --git a/src/canvas.c b/src/canvas.c
index e24a404..9c860ae 100644
--- a/src/canvas.c
+++ b/src/canvas.c
@@ -17,11 +17,11 @@ uint8 is_drawing;
 #define COORD(x,y)  ((x) + (y) * c->w)
 #define HNEXT(i)    (i == HISTLENGTH-1 ? 0 : i+1)
 #define HPREV(i)    (i == 0 ? HISTLENGTH-1 : i-1)
+#define PIXEL(x, y) c->cells[c->layer_arr_cnt * c->cur_frame + c->cur_layer][COORD(x,y)]
 
 typedef struct Layer Layer;
 
-static Layer   *layer_create(uint, uint, uint);
-static void     layer_destroy(Layer *);
+static uint8    canvas_add_cells(Canvas *, uint);
 static uint     canvas_fill_bfs(Canvas *, int, int, uint, uint);
 static uint     canvas_blend_color(uint, uint);
 static void     canvas_point_redraw(Canvas *, long int, long int);
@@ -47,6 +47,7 @@ canvas_coord_get(Canvas *c, long int tx, long int ty, long int *x, long int *y)
 Canvas *
 canvas_init(uint w, uint h, void *ren)
 {
+	/* TODO: Add propper deffer */
 	Canvas *c;
 	int i, j;
 	SDL_Rect dest;
@@ -61,26 +62,37 @@ canvas_init(uint w, uint h, void *ren)
 	c->cur_frame = 0;
 	c->zoom = 1;
 	c->proj_path = NULL;
-	c->layer_arr_cnt = 1;
+	c->layer_arr_cnt = 0;
 	c->layer_arr_sz = 1;
 	/* TEMP: TODO */
-	c->frame_arr_cnt = 4;
-	c->frame_arr_sz = 4;
+	c->frame_arr_cnt = 0;
+	c->frame_arr_sz = 1;
+	c->cell_arr_sz = 1;
 	c->x = c->y = 0;
 	c->hist_i = c->hist_s = c->hist_isend = 0;
 	c->hist_e = HNEXT(c->hist_s);
 	c->layers = malloc(sizeof(*(c->layers)));
+	c->frames = malloc(sizeof(*(c->frames)));
 	c->temp_pix = malloc(w * h * sizeof(* c->temp_pix));
+	c->cells = malloc(sizeof(* c->cells));
 	for (i = 0; i < HISTLENGTH; ++i)
 		c->history[i].type = ACT_NULL;
 	c->pres_pix = malloc(w * h * sizeof(* c->pres_pix));
-
-	c->layers[0] = layer_create(w, h, 1);
+	if (c->cells != NULL)
+		c->cells[0] = NULL;
+	else
+		return NULL;
 
 	for (i = 0; i < c->w * c->h; ++i)
 		c->pres_pix[i] = 0;
 
-    if (c->layers[0] == NULL) {
+    if (canvas_add_frame(c, 0)) {
+        fprintf(stderr, "Error while creating frame");
+        free(c);
+        return NULL;
+    }
+
+    if (canvas_add_layer(c, 0)) {
         fprintf(stderr, "Error while creating layer");
         free(c);
         return NULL;
@@ -172,7 +184,7 @@ canvas_open1(FILE *fp, void *ren)
 	Canvas *c;
 	fscanf(fp, "%u;%u;", &w, &h);
 	c = canvas_init(w, h, ren);
-	fread(c->layers[0]->pix, sizeof *(c->layers[0]->pix), c->w*c->h, fp);
+	fread(c->cells[0], sizeof *(c->cells[0]), c->w*c->h, fp);
 
 	return c;
 }
@@ -188,13 +200,13 @@ canvas_open2(FILE *fp, void *ren)
 	for (i = 1; i < lc; i++)
 		canvas_add_layer(c, lc);
 	for (i = 0; i < lc; i++) {
-		fread(c->layers[i]->name, sizeof *(c->layers[i]->name), LAYERNAMELEN, fp);
-		c->layers[i]->name[LAYERNAMELEN] = '\0';
+		fread(c->layers[i].name, sizeof *(c->layers[i].name), LAYERNAMELEN, fp);
+		c->layers[i].name[LAYERNAMELEN] = '\0';
 		fscanf(fp, ";");
 	}
 
 	for (i = 0; i < lc; i++)
-		fread(c->layers[i]->pix, sizeof *(c->layers[i]->pix), c->w*c->h, fp);
+		fread(c->cells[i], sizeof *(c->cells[i]), c->w*c->h, fp);
 
 	return c;
 }
@@ -224,7 +236,7 @@ canvas_import_png(const char *path, void *ren)
     rect.w = sur->w;
     rect.h = sur->h;
     SDL_RenderReadPixels(ren, &rect, SDL_PIXELFORMAT_RGBA8888,
-            c->layers[0]->pix, (sizeof *(c->layers[0]->pix)) * c->w);
+			c->cells[0], (sizeof *(c->cells[0])) * c->w);
     SDL_SetRenderTarget(ren, NULL);
 
     SDL_DestroyTexture(tex);
@@ -242,11 +254,15 @@ canvas_destroy(Canvas *c)
 	SDL_DestroyTexture(c->pres);
 	SDL_DestroyTexture(c->back);
 	SDL_DestroyTexture(c->half_pres);
-	for (i = 0; i < c->layer_arr_cnt; ++i)
-		layer_destroy(c->layers[i]);
+	if (c->cells != NULL) {
+		for (i = 0; i < c->layer_arr_cnt * c->frame_arr_cnt; i++)
+			free(c->cells[i]);
+		free(c->cells);
+	}
 	for (i = 0; i < HISTLENGTH; ++i)
 		action_remove(&c->history[i]);
 	free(c->layers);
+	free(c->frames);
 	free(c->temp_pix);
 	free(c->pres_pix);
 	free(c->proj_path);
@@ -307,7 +323,7 @@ canvas_point_draw(Canvas *c, long int x, long int y)
 	unsigned int oldcol, newcol;
 	int i;
 /* TODO: better */
-	oldcol = c->layers[c->cur_layer]->pix[COORD(x,y)];
+	oldcol = PIXEL(x, y);
 	newcol = COLTOINTA(
 	        ((Palette *)def_palette)->clist[c->cur_col].r,
 	        ((Palette *)def_palette)->clist[c->cur_col].g,
@@ -324,16 +340,16 @@ canvas_point_draw(Canvas *c, long int x, long int y)
 	if (tool_array[tool_cur] == TOOL_TYPE_ERASER) {
 		c->temp_pix[c->temp_cnt].x = x;
 		c->temp_pix[c->temp_cnt].y = y;
-		c->temp_pix[c->temp_cnt++].c = c->layers[c->cur_layer]->pix[COORD(x,y)];
-		c->layers[c->cur_layer]->pix[COORD(x,y)] = 0;
+		c->temp_pix[c->temp_cnt++].c = PIXEL(x, y);
+		PIXEL(x, y) = 0;
 		canvas_point_redraw(c, x, y);
 		SDL_SetRenderDrawColor(ren, INTTOCOLA(c->pres_pix[COORD(x,y)]));
 		SDL_RenderDrawPoint(ren, x, y);
 	} else if (tool_array[tool_cur] == TOOL_TYPE_PENCIL) {
 		c->temp_pix[c->temp_cnt].x = x;
 		c->temp_pix[c->temp_cnt].y = y;
-		c->temp_pix[c->temp_cnt++].c = c->layers[c->cur_layer]->pix[COORD(x,y)];
-		c->layers[c->cur_layer]->pix[COORD(x,y)] = COLTOINTA(
+		c->temp_pix[c->temp_cnt++].c = PIXEL(x, y);
+		PIXEL(x, y) = COLTOINTA(
 		        ((Palette *)def_palette)->clist[c->cur_col].r,
 		        ((Palette *)def_palette)->clist[c->cur_col].g,
 		        ((Palette *)def_palette)->clist[c->cur_col].b,
@@ -363,7 +379,7 @@ canvas_mousel_down(Canvas *c, long int x, long int y)
 	switch (tool_array[tool_cur]) {
 	case TOOL_TYPE_CPICKER:
 		for (i = 0; i < ((Palette *)def_palette)->num; ++i) {
-			if (c->layers[c->cur_layer]->pix[COORD(x,y)] == COLTOINTA(
+			if (PIXEL(x, y) == COLTOINTA(
 			        ((Palette *)def_palette)->clist[i].r,
 			        ((Palette *)def_palette)->clist[i].g,
 			        ((Palette *)def_palette)->clist[i].b,
@@ -436,11 +452,11 @@ canvas_save(Canvas *c, const char *path, short int s)
 
 	fprintf(fp, "2;%u;%u;%u;", c->w, c->h, c->layer_arr_cnt);
 	for (i = 0; i < c->layer_arr_cnt; i++) {
-		fwrite(c->layers[i]->name, sizeof *(c->layers[i]->name), LAYERNAMELEN, fp);
+		fwrite(c->layers[i].name, sizeof *(c->layers[i].name), LAYERNAMELEN, fp);
 		fprintf(fp, ";");
 	}
 	for (i = 0; i < c->layer_arr_cnt; i++)
-		fwrite(c->layers[i]->pix, sizeof *(c->layers[i]->pix), c->w*c->h, fp);
+		fwrite(c->cells[i], sizeof *(c->cells[i]), c->w*c->h, fp);
 	fclose(fp);
 	if (s)
 		canvas_set_proj_path(c, path);
@@ -482,12 +498,16 @@ canvas_export_png(Canvas *c, const char *path, void *ren)
 	return ret;
 }
 
-void
+uint8
 canvas_add_layer(Canvas *c, uint pos)
 {
-	int i;
+	int i, j;
+	uint *u;
 	if (c == NULL)
-		return;
+		return 1;
+
+	if (canvas_add_cells(c, c->frame_arr_cnt))
+		return 1;
 	pos = (pos > c->layer_arr_cnt) ? c->layer_arr_cnt : pos;
 
 	if (c->layer_arr_cnt == c->layer_arr_sz) {
@@ -500,7 +520,77 @@ canvas_add_layer(Canvas *c, uint pos)
 	}
 
 	c->layer_arr_cnt++;
-	c->layers[pos] = layer_create(c->w, c->h, c->layer_arr_cnt);
+	c->layers[pos].visible = 1;
+	strcpy(c->layers[pos].name, "Layer ");
+	i = 10000000;
+	while (i) {
+		if (c->layer_arr_cnt / i) {
+			c->layers[pos].name[strlen(c->layers[pos].name)+1] = '\0';
+			c->layers[pos].name[strlen(c->layers[pos].name)] = ((c->layer_arr_cnt/i)%10) + '0';
+		}
+		i /= 10;
+	}
+
+	if (c->layer_arr_cnt > 1) {
+		for (i = c->frame_arr_cnt - 1; i >= 0; i--) {
+			for (j = c->layer_arr_cnt - 2; j >= 0; j--) {
+				if (j >= pos) {
+					u = c->cells[i*c->layer_arr_cnt - i + j];
+					c->cells[i*c->layer_arr_cnt - i + j] = c->cells[i*c->layer_arr_cnt + j + 1];
+					c->cells[i*c->layer_arr_cnt + j + 1] = u;
+				} else {
+					u = c->cells[i*c->layer_arr_cnt - i + j];
+					c->cells[i*c->layer_arr_cnt - i + j] = c->cells[i*c->layer_arr_cnt + j];
+					c->cells[i*c->layer_arr_cnt + j] = u;
+				}
+			}
+		}
+	}
+
+	for (i = 0; i < c->frame_arr_cnt; i++)
+		for (j = 0; j < c->w * c->h; j++)
+			c->cells[i*c->layer_arr_cnt + pos][j] = 0;
+	return 0;
+}
+
+uint8
+canvas_add_frame(Canvas *c, uint pos)
+{
+	int i, j;
+	uint *u;
+	if (c == NULL)
+		return 1;
+
+	if (canvas_add_cells(c, c->layer_arr_cnt))
+		return 1;
+	pos = (pos > c->frame_arr_cnt) ? c->frame_arr_cnt : pos;
+
+	if (c->frame_arr_cnt == c->frame_arr_sz) {
+		c->frame_arr_sz *= 2;
+		c->frames = realloc(c->frames, c->frame_arr_sz * sizeof(* c->frames));
+	}
+
+	for (i = c->frame_arr_cnt; i > pos; i--) {
+		c->frames[i] = c->frames[i-1];
+	}
+
+	c->frame_arr_cnt++;
+	c->frames[pos].state = 0;
+
+	if (c->layer_arr_cnt && c->frame_arr_cnt > 1) {
+		for (i = c->frame_arr_cnt - 2; i >= pos; i--) {
+			for (j = c->layer_arr_cnt - 1; j >= 0; j--) {
+				u = c->cells[i * c->layer_arr_cnt + j];
+				c->cells[i*c->layer_arr_cnt + j] = c->cells[(i+1)*c->layer_arr_cnt + j];
+				c->cells[(i+1)*c->layer_arr_cnt + j] = u;
+			}
+		}
+	}
+
+	for (i = 0; i < c->layer_arr_cnt; i++)
+		for (j = 0; j < c->w * c->h; j++)
+			c->cells[pos*c->layer_arr_cnt + i][j] = 0;
+	return 0;
 }
 
 void
@@ -525,6 +615,16 @@ canvas_change_layer(Canvas *c, unsigned int l)
 		c->cur_layer = l;
 }
 
+void
+canvas_change_frame(Canvas *c, unsigned int l)
+{
+	if (l < c->frame_arr_cnt && l != c->cur_frame) {
+		c->cur_frame = l;
+		canvas_refresh(c);
+		ui_redraw_panel(UI_PANELTYPE_CANVAS);
+	}
+}
+
 
 static void
 canvas_set_proj_path(Canvas *c, const char *path)
@@ -544,46 +644,34 @@ canvas_set_proj_path(Canvas *c, const char *path)
 	free(title);
 }
 
-static Layer *
-layer_create(uint w, uint h, uint cnt)
-{
-	Layer *res;
-	int i;
-	/* TODO: better error handling and maybe allocating together? */
-	res = malloc(sizeof *res);
-	if (res == NULL)
-		return NULL;
-
-	res->visible = 1;
-	res->pix = malloc(h * w * sizeof(* res->pix));
-	if (res->pix == NULL) {
-		fprintf(stderr, "Error creating layer\n");
-		layer_destroy(res);
-		return NULL;
+static uint8
+canvas_add_cells(Canvas * c, uint k)
+{
+	if (c == NULL) return 1;
+	uint **cells;
+	uint i, s;
+	s = c->layer_arr_cnt * c->frame_arr_cnt;
+	while (s + k > c->cell_arr_sz) {
+		cells = realloc(c->cells, 2 * c->cell_arr_sz * sizeof(* cells));
+		if (cells == NULL) return 1;
+		c->cells = cells;
+		c->cell_arr_sz *= 2;
 	}
 
-	for (i = 0; i < w*h; ++i)
-		res->pix[i] = 0;
-	strcpy(res->name, "Layer ");
-	i = 10000000;
-	while (i) {
-		if (cnt / i) {
-			res->name[strlen(res->name)+1] = '\0';
-			res->name[strlen(res->name)] = ((cnt/i)%10) + '0';
+	cells = c->cells;
+
+	for (i = s; i < c->cell_arr_sz; i++)
+		cells[i] = NULL;
+	for (i = s; i < s + k; i++) {
+		cells[i] = malloc(c->w * c->h * sizeof(* cells[i]));
+		if (cells[i] == NULL) {
+			for (k = s; k < i; k++)
+				free(cells[i]);
+			return 1;
 		}
-		i /= 10;
 	}
 
-
-	return res;
-}
-
-static void
-layer_destroy(Layer *lay)
-{
-    if (lay->pix != NULL)
-        free(lay->pix);
-    free(lay);
+	return 0;
 }
 
 static uint
@@ -599,35 +687,35 @@ canvas_fill_bfs(Canvas *c, int x, int y, uint oldcol, uint newcol)
 
 	c->temp_pix[0].x = x;
 	c->temp_pix[0].y = y;
-	c->layers[c->cur_layer]->pix[COORD(x,y)] = newcol;
+	PIXEL(x, y) = newcol;
 
     for (; pnt != &(c->temp_pix)[c->temp_cnt]; ++pnt) {
 		pnt[0].c = oldcol;
         x = pnt[0].x;
         y = pnt[0].y;
-        if (x > 0 && c->layers[c->cur_layer]->pix[COORD(x - 1,y)] == oldcol) {
-            c->layers[c->cur_layer]->pix[COORD(x-1, y)] = newcol;
+        if (x > 0 && PIXEL(x-1, y) == oldcol) {
+            PIXEL(x-1, y) = newcol;
             c->temp_pix[c->temp_cnt].x = x - 1;
             c->temp_pix[c->temp_cnt].y = y;
             ++c->temp_cnt;
         }
 
-        if (x < c->w - 1 && c->layers[c->cur_layer]->pix[COORD(x+1, y)] == oldcol) {
-            c->layers[c->cur_layer]->pix[COORD(x+1, y)] = newcol;
+        if (x < c->w - 1 && PIXEL(x+1, y) == oldcol) {
+            PIXEL(x+1, y) = newcol;
             c->temp_pix[c->temp_cnt].x = x + 1;
             c->temp_pix[c->temp_cnt].y = y;
             ++c->temp_cnt;
         }
 
-        if (y > 0 && c->layers[c->cur_layer]->pix[COORD(x, y-1)] == oldcol) {
-            c->layers[c->cur_layer]->pix[COORD(x, y-1)] = newcol;
+        if (y > 0 && PIXEL(x, y-1) == oldcol) {
+            PIXEL(x, y-1) = newcol;
             c->temp_pix[c->temp_cnt].x = x;
             c->temp_pix[c->temp_cnt].y = y - 1;
             ++c->temp_cnt;
         }
 
-        if (y < c->h - 1 && c->layers[c->cur_layer]->pix[COORD(x, y+1)] == oldcol) {
-            c->layers[c->cur_layer]->pix[COORD(x, y+1)] = newcol;
+        if (y < c->h - 1 && PIXEL(x, y+1) == oldcol) {
+            PIXEL(x, y+1) = newcol;
             c->temp_pix[c->temp_cnt].x = x;
             c->temp_pix[c->temp_cnt].y = y + 1;
             ++c->temp_cnt;
@@ -652,8 +740,8 @@ canvas_point_redraw(Canvas *c, long int x, long int y)
 	c->pres_pix[COORD(x, y)] = 0;
 
 	for (i = 0; i < c->layer_arr_cnt; i++)
-		if (c->layers[i]->visible)
-			c->pres_pix[COORD(x, y)] = canvas_blend_color(c->pres_pix[COORD(x, y)], c->layers[i]->pix[COORD(x, y)]);
+		if (c->layers[i].visible)
+			c->pres_pix[COORD(x, y)] = canvas_blend_color(c->pres_pix[COORD(x, y)], c->cells[c->layer_arr_cnt * c->cur_frame + i][COORD(x, y)]);
 }
 
 static void
@@ -662,7 +750,8 @@ action_do(Canvas *c)
 	action_remove(&c->history[c->hist_i]);
 	c->history[c->hist_i].type = ACT_PIXELSCOLORS;
 	c->history[c->hist_i].act.px.cnt = c->temp_cnt;
-	c->history[c->hist_i].act.px.lyr = c->cur_layer;
+	c->history[c->hist_i].act.px.layer = c->cur_layer;
+	c->history[c->hist_i].act.px.frame = c->cur_frame;
 	c->history[c->hist_i].act.px.pix = malloc(c->temp_cnt * sizeof(struct action_pixcol));
 	memcpy(c->history[c->hist_i].act.px.pix, c->temp_pix, c->temp_cnt * sizeof(struct action_pixcol));
 	if (c->hist_i == c->hist_e)
@@ -725,15 +814,16 @@ action_pixcols_redo(Canvas *c, Action *a)
 {
 	struct action_pixcol *pnt, *end;
 	unsigned int col;
-	canvas_change_layer(c, a->act.px.lyr);
+	canvas_change_layer(c, a->act.px.layer);
+	canvas_change_frame(c, a->act.px.frame);
 	ui_redraw_panel(UI_PANELTYPE_TIMELINE);
 	pnt = a->act.px.pix;
 	end = &pnt[a->act.px.cnt];
 
 	SDL_SetRenderTarget(ren, c->half_pres);
 	for (; pnt != end; pnt++) {
-		col = c->layers[c->cur_layer]->pix[COORD(pnt->x, pnt->y)];
-		c->layers[c->cur_layer]->pix[COORD(pnt->x, pnt->y)] = pnt->c;
+		col = PIXEL(pnt->x, pnt->y);
+		PIXEL(pnt->x, pnt->y) = pnt->c;
 		pnt->c = col;
 		canvas_point_redraw(c, pnt->x, pnt->y);
 		SDL_SetRenderDrawColor(ren, INTTOCOLA(c->pres_pix[COORD(pnt->x, pnt->y)]));
diff --git a/src/canvas.h b/src/canvas.h
index 58f20b7..f76f0a1 100644
--- a/src/canvas.h
+++ b/src/canvas.h
@@ -5,22 +5,24 @@
 #define LAYERNAMELEN 30
 
 struct Layer {
-	unsigned int *pix;
 	unsigned int visible;
 	char name[LAYERNAMELEN+1];
 };
 
 struct Frame {
-	unsigned int state;
+	unsigned char state;
 };
 
 struct Canvas {
 	unsigned int layer_arr_cnt, layer_arr_sz;
 	unsigned int frame_arr_cnt, frame_arr_sz;
+	unsigned int cell_arr_cnt, cell_arr_sz;
 	unsigned int cur_col, cur_layer, cur_frame;
 	unsigned int w, h, zoom;
 	char *proj_path;
-	struct Layer **layers;
+	struct Layer *layers;
+	struct Frame *frames;
+	unsigned int **cells;
 	int x, y;
 	void *back, *pres, *half_pres;
 	struct action_pixcol *temp_pix;
@@ -51,11 +53,13 @@ void            canvas_point_draw(Canvas *, long int, long int);
 void            canvas_mousel_up(Canvas *);
 void            canvas_mousel_down(Canvas *, long int, long int);
 void            canvas_mouse_move(Canvas *, long int, long int);
-void            canvas_add_layer(Canvas *, unsigned int);
+unsigned char   canvas_add_layer(Canvas *, unsigned int);
+unsigned char   canvas_add_frame(Canvas *, unsigned int);
 void            canvas_move_x(Canvas *, long int);
 void            canvas_move_y(Canvas *, long int);
 void            canvas_refresh(Canvas *);
 void            canvas_change_layer(Canvas *, unsigned int);
+void            canvas_change_frame(Canvas *, unsigned int);
 unsigned char   canvas_save(Canvas *, const char *, short int);
 unsigned char   canvas_export_png(Canvas *, const char *, void *);
 void            action_undo(Canvas *);
diff --git a/src/cdraw.c b/src/cdraw.c
index 2e0097a..37ad773 100644
--- a/src/cdraw.c
+++ b/src/cdraw.c
@@ -204,7 +204,7 @@ main_event_handle()
 				switch(event.key.keysym.sym) {
 				case SDLK_BACKSPACE:
 					if (cur_canvas == NULL) break;
-					text = cur_canvas->layers[cur_canvas->a]->name;
+					text = cur_canvas->layers[cur_canvas->a].name;
 					if (strlen(text))
 						text[strlen(text)-1] = '\0';
 					ui_redraw_panel(UI_PANELTYPE_TIMELINE);
@@ -217,7 +217,7 @@ main_event_handle()
 				break;
 			case SDL_TEXTINPUT:
 				if (cur_canvas == NULL) break;
-				text = cur_canvas->layers[cur_canvas->a]->name;
+				text = cur_canvas->layers[cur_canvas->a].name;
 				if (strlen(text) >= LAYERNAMELEN) break;
 				strncat(text, event.text.text, LAYERNAMELEN - strlen(text));
 				ui_redraw_panel(UI_PANELTYPE_TIMELINE);
@@ -258,4 +258,6 @@ main(int argc, char *argv[])
         if (ltick < fdif)
             SDL_Delay(fdif-ltick);
     }
+
+	return 0;
 }
diff --git a/src/cdraw.h b/src/cdraw.h
index 2a6710a..70e6350 100644
--- a/src/cdraw.h
+++ b/src/cdraw.h
@@ -35,5 +35,7 @@ void            user_canvas_redo(const Arg *);
 void            user_tool_change(const Arg *);
 void            user_layer_add(const Arg *);
 void            user_layer_chng(const Arg *);
+void            user_frame_add(const Arg *);
+void            user_frame_chng(const Arg *);
 void            user_testing_reload_tex(const Arg *);
 void            user_debug_mem_show(const Arg *);
diff --git a/src/ui.c b/src/ui.c
index d126215..52ce31e 100644
--- a/src/ui.c
+++ b/src/ui.c
@@ -303,15 +303,19 @@ ui_mousel_down(int x, int y)
 		y -= 1;
 		if (y < 0 || y >= cur_canvas->layer_arr_cnt) break;
 		if (x <= 22) {
-			cur_canvas->layers[cur_canvas->layer_arr_cnt-1-y]->visible ^= 1;
+			cur_canvas->layers[cur_canvas->layer_arr_cnt-1-y].visible ^= 1;
 			canvas_refresh(cur_canvas);
 			ui_redraw_panel(UI_PANELTYPE_TIMELINE);
 			ui_redraw_panel(UI_PANELTYPE_CANVAS);
-		} else if (x <= 162) {
+		} else if (x <= 162 + cur_canvas->frame_arr_cnt*22) {
 			y = cur_canvas->layer_arr_cnt-y-1;
 			canvas_change_layer(cur_canvas, y);
 			ui_redraw_panel(UI_PANELTYPE_TIMELINE);
 		}
+		x = (x - 162) / 22;
+		if (x < 0 || x >= cur_canvas->frame_arr_cnt) break;
+		canvas_change_frame(cur_canvas, x);
+		ui_redraw_panel(UI_PANELTYPE_TIMELINE);
 		break;
 	}
 }
@@ -713,10 +717,21 @@ ui_panel_timeline_redraw(UIPanelTimeline *p)
 	SDL_RenderClear(ren);
 	if (cur_canvas != NULL) {
 		/* Draw selected */
+		SDL_SetRenderDrawColor(ren, INTTOCOLA(0x92a1b9ff));
 		dest.x = 2;
+		dest.h = 24;
+		dest.y = (cur_canvas->layer_arr_cnt - cur_canvas->cur_layer) * 22;
+		dest.w = 162 + 22 * cur_canvas->frame_arr_cnt;
+		SDL_RenderFillRect(ren, &dest);
+		dest.x = 164 + cur_canvas->cur_frame * 22;
+		dest.y = 2;
+		dest.h = (cur_canvas->layer_arr_cnt + 1) * 22 - 2;
+		dest.w = 24;
+		SDL_RenderFillRect(ren, &dest);
+		dest.x = 166 + cur_canvas->cur_frame * 22;
+		dest.y = (cur_canvas->layer_arr_cnt - cur_canvas->cur_layer) * 22 + 2;
 		dest.h = 20;
-		dest.w = 162;
-		dest.y = (cur_canvas->layer_arr_cnt - 1 - cur_canvas->cur_layer) * 22 + 24;
+		dest.w = 20;
 		SDL_SetRenderDrawColor(ren, INTTOCOLA(0x657392ff));
 		SDL_RenderFillRect(ren, &dest);
 
@@ -745,7 +760,7 @@ ui_panel_timeline_redraw(UIPanelTimeline *p)
 			SDL_SetRenderDrawColor(ren, INTTOCOLA(0x000000ff));
 			dest.y = dest.x = 2;
 			dest.w = dest.h = 20;
-			if (cur_canvas->layers[i]->visible)
+			if (cur_canvas->layers[i].visible)
 				SDL_RenderCopy(ren, main_ui->theme, &ui_timeline_rect[0], &dest);
 			else
 				SDL_RenderCopy(ren, main_ui->theme, &ui_timeline_rect[1], &dest);
@@ -754,7 +769,7 @@ ui_panel_timeline_redraw(UIPanelTimeline *p)
 			dest.w = 140;
 			dest.h = 24;
 			SDL_RenderSetViewport(ren, &dest);
-			SBDF_SDLPrint(ren, font, cur_canvas->layers[i]->name, 2, 17);
+			SBDF_SDLPrint(ren, font, cur_canvas->layers[i].name, 2, 17);
 		}
 
 		/* Draw Frame row */
@@ -796,6 +811,26 @@ ui_panel_timeline_redraw(UIPanelTimeline *p)
 
 			SBDF_SDLPrint(ren, font, text, j, 17);
 		}
+
+		/* Draw cell table */
+		SDL_SetRenderDrawColor(ren, INTTOCOLA(0x131313ff));
+		for (i = 0; i < cur_canvas->layer_arr_cnt; i++) {
+			for (j = 0; j < cur_canvas->frame_arr_cnt; j++) {
+				dest.y = 28 + i * 22;
+				dest.x = 170 + j * 22;
+				dest.h = 24;
+				dest.w = 24;
+				SDL_RenderSetViewport(ren, &dest);
+				SDL_RenderDrawLine(ren, 2, 0, 21, 0);
+				SDL_RenderDrawLine(ren, 2, 1, 21, 1);
+				SDL_RenderDrawLine(ren, 0, 2, 0, 21);
+				SDL_RenderDrawLine(ren, 1, 2, 1, 21);
+				SDL_RenderDrawLine(ren, 2, 22, 21, 22);
+				SDL_RenderDrawLine(ren, 2, 23, 21, 23);
+				SDL_RenderDrawLine(ren, 22, 2, 22, 21);
+				SDL_RenderDrawLine(ren, 23, 2, 23, 21);
+			}
+		}
 	}
 	SDL_RenderSetViewport(ren, NULL);
 	ui_panel_draw_frame(&p->head);
diff --git a/src/user.c b/src/user.c
index 9d1b437..7469708 100644
--- a/src/user.c
+++ b/src/user.c
@@ -168,7 +168,7 @@ void
 user_layer_add(const Arg *x)
 {
 	if (cur_canvas == NULL) return;
-	canvas_add_layer(cur_canvas, cur_canvas->cur_layer+1);
+	if (canvas_add_layer(cur_canvas, cur_canvas->cur_layer+1)) return;
 	cur_canvas->cur_layer++;
 	ui_redraw_panel(UI_PANELTYPE_CANVAS);
 	ui_redraw_panel(UI_PANELTYPE_TIMELINE);
@@ -183,6 +183,29 @@ user_layer_chng(const Arg *x)
 }
 
 void
+user_frame_add(const Arg *x)
+{
+	int i;
+	if (cur_canvas == NULL) return;
+	if (canvas_add_frame(cur_canvas, cur_canvas->cur_frame+1)) return;
+	for (i = 0; i < cur_canvas->layer_arr_cnt; i++)
+		memcpy(cur_canvas->cells[(cur_canvas->cur_frame + 1) * cur_canvas->layer_arr_cnt+i],
+		    cur_canvas->cells[cur_canvas->cur_frame * cur_canvas->layer_arr_cnt+i],
+		    cur_canvas->w * cur_canvas->h * sizeof *cur_canvas->cells[0]);
+	canvas_change_frame(cur_canvas, cur_canvas->cur_frame+1);
+	ui_redraw_panel(UI_PANELTYPE_CANVAS);
+	ui_redraw_panel(UI_PANELTYPE_TIMELINE);
+}
+
+void
+user_frame_chng(const Arg *x)
+{
+	if (cur_canvas == NULL) return;
+	canvas_change_frame(cur_canvas, (cur_canvas->cur_frame + (unsigned int)cur_canvas->frame_arr_cnt + (x->i % (int)cur_canvas->frame_arr_cnt)) % cur_canvas->frame_arr_cnt);
+	ui_redraw_panel(UI_PANELTYPE_TIMELINE);
+}
+
+void
 user_debug_mem_show(const Arg *x)
 {
 	fprintf(stdout, "DEBUG INFO:\n");