From f925e225619eadcc97c0bed12b350cb31cf3d1bb Mon Sep 17 00:00:00 2001
From: David Capello <david@igara.com>
Date: Tue, 30 Aug 2022 17:35:39 -0300
Subject: [PATCH] Don't encourage the usage of layerIndex param in
 ExportSpriteSheetParams

Until we don't have a standarized way to indicate layers by indexes.
---
 src/app/commands/cmd_export_sprite_sheet.cpp | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/app/commands/cmd_export_sprite_sheet.cpp b/src/app/commands/cmd_export_sprite_sheet.cpp
index fe5c58ee4..c96380614 100644
--- a/src/app/commands/cmd_export_sprite_sheet.cpp
+++ b/src/app/commands/cmd_export_sprite_sheet.cpp
@@ -76,7 +76,12 @@ struct ExportSpriteSheetParams : public NewParams {
   Param<bool> mergeDuplicates { this, false, "mergeDuplicates" };
   Param<bool> openGenerated { this, false, "openGenerated" };
   Param<std::string> layer { this, std::string(), "layer" };
-  Param<int> layerIndex { this, -1, "layerIndex" };
+  // TODO The layerIndex parameter is for internal use only, layers
+  //      are counted in the same order as they are displayed in the
+  //      Timeline or in the Export Sprite Sheet combobox. But this
+  //      index is different to the one specified in the .aseprite
+  //      file spec (where layers are counted from bottom to top).
+  Param<int> layerIndex { this, -1, "_layerIndex" };
   Param<std::string> tag { this, std::string(), "tag" };
   Param<bool> splitLayers { this, false, "splitLayers" };
   Param<bool> splitTags { this, false, "splitTags" };