From: Tae-Hwan Kim <the81.kim@samsung.com>
Date: Sun, 12 May 2013 09:42:24 +0000 (+0900)
Subject: [Genlist] Add comments
X-Git-Tag: accepted/tizen/20131029.210118~302
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=bf01784901a544a4354f7a2ce2a8713e3c1f6f89;p=profile%2Fivi%2Fefl-theme-tizen.git

[Genlist] Add comments

Change-Id: Ib467d416004931f4da647a37ce2b201b9fb410a6
---

diff --git a/themes/widgets/genlist/genlist_normal.edc b/themes/widgets/genlist/genlist_normal.edc
index 7c148d8..aae228c 100644
--- a/themes/widgets/genlist/genlist_normal.edc
+++ b/themes/widgets/genlist/genlist_normal.edc
@@ -2517,22 +2517,29 @@
          GENLIST_PART_PADDING_TOP_2LINE
          GENLIST_PART_PADDING_BOTTOM_2LINE
 
-         GENLIST_PART_LISTITEM_TB_("elm.text.1", GENLIST_HEIGHT_2LINE_MAIN,
-            "elm.padding.left", "elm.padding.icon1.left",
-            "elm.padding.top", "elm.text.3",
+         GENLIST_PART_LISTITEM_LRT_TB("elm.text.1", GENLIST_HEIGHT_2LINE_MAIN,
+            "elm.padding.left", "elm.padding.icon1.left", "elm.padding.top",
          )
          GENLIST_PART_PADDING_R("elm.padding.icon1.left", "elm.icon.1")
          GENLIST_PART_ICON_SMALL_RTB("elm.icon.1",
-            "elm.padding.text4.left", "elm.padding.top", "elm.text.3")
+            "elm.padding.text4.left", "elm.padding.top", "elm.text.2"
+         )
          GENLIST_PART_PADDING_R("elm.padding.text4.left", "elm.text.4")
+
+         // FIXME: This part depends on elm.text.2 as bottom part.
+         // So if elm.text.3 (the other bottom part) size is changed,
+         // It can break this part's height.
          GENLIST_PART_SUBITEM_RTB_TB("elm.text.4", GENLIST_HEIGHT_2LINE_MAIN,
-            "elm.padding.right", "elm.padding.top", "elm.text.3")
+            "elm.padding.right", "elm.padding.top", "elm.text.2"
+         )
 
-         GENLIST_PART_SUBITEM_LRB_TB("elm.text.3", GENLIST_HEIGHT_2LINE_SUB,
-            "elm.padding.left", "elm.padding.text2.left", "elm.padding.bottom")
+         GENLIST_PART_SUBITEM_TB_("elm.text.3", GENLIST_HEIGHT_2LINE_SUB,
+            "elm.padding.left", "elm.padding.text2.left", "elm.text.1", "elm.padding.bottom"
+         )
          GENLIST_PART_PADDING_R("elm.padding.text2.left", "elm.text.2")
          GENLIST_PART_SUBITEM_RTB_TB("elm.text.2", GENLIST_HEIGHT_2LINE_SUB,
-            "elm.padding.right", "elm.text.1", "elm.padding.bottom")
+            "elm.padding.right", "elm.text.1", "elm.padding.bottom"
+         )
          GENLIST_PART_FLIP
          GENLIST_PART_FOCUS
       }