diff options
author | Jose Antonio Marquez | 2012-06-21 12:51:14 -0700 |
---|---|---|
committer | Jose Antonio Marquez | 2012-06-21 12:51:14 -0700 |
commit | 4b186e0c3f97f704d1ac671a63cd02084e19fe0e (patch) | |
tree | 9cbfd72644568daae9c5b9f43126ddc214d260dc /js/components | |
parent | 28aa1276ffb6cefca7f7d58906cf8380fc6253eb (diff) | |
download | ninja-4b186e0c3f97f704d1ac671a63cd02084e19fe0e.tar.gz |
Fixing reference issue for toolbar fill container
The wrapper container was not being referenced, caused some toggling visibility issues.
Diffstat (limited to 'js/components')
-rwxr-xr-x | js/components/tools-properties/shape-properties.reel/shape-properties.html | 3 | ||||
-rwxr-xr-x | js/components/tools-properties/shape-properties.reel/shape-properties.js | 8 |
2 files changed, 6 insertions, 5 deletions
diff --git a/js/components/tools-properties/shape-properties.reel/shape-properties.html b/js/components/tools-properties/shape-properties.reel/shape-properties.html index 22ea7431..5f93b556 100755 --- a/js/components/tools-properties/shape-properties.reel/shape-properties.html +++ b/js/components/tools-properties/shape-properties.reel/shape-properties.html | |||
@@ -112,6 +112,7 @@ | |||
112 | "_fillColorCtrlIcon": {"#": "fillColorCtrlIcon"}, | 112 | "_fillColorCtrlIcon": {"#": "fillColorCtrlIcon"}, |
113 | "_fillColorCtrl": {"#": "fillColorCtrl"}, | 113 | "_fillColorCtrl": {"#": "fillColorCtrl"}, |
114 | "_strokeColorCtrl": {"#": "strokeColorCtrl"}, | 114 | "_strokeColorCtrl": {"#": "strokeColorCtrl"}, |
115 | "_fillColorCtrlContainer": {"#": "fillColorCtrlContainer"}, | ||
115 | 116 | ||
116 | "_strokeSize": {"@": "_strokeSize1"}, | 117 | "_strokeSize": {"@": "_strokeSize1"}, |
117 | "ovalProperties": {"@": "ovalProperties1"}, | 118 | "ovalProperties": {"@": "ovalProperties1"}, |
@@ -137,7 +138,7 @@ | |||
137 | <div class="toolColorChipIcon PencilTool"></div> | 138 | <div class="toolColorChipIcon PencilTool"></div> |
138 | <div class="toolColorChipContainer"><div data-montage-id="strokeColorCtrl" class="toolColorChipCtrl"></div></div> | 139 | <div class="toolColorChipContainer"><div data-montage-id="strokeColorCtrl" class="toolColorChipCtrl"></div></div> |
139 | <div data-montage-id="fillColorCtrlIcon" class="toolColorChipIcon FillTool"></div> | 140 | <div data-montage-id="fillColorCtrlIcon" class="toolColorChipIcon FillTool"></div> |
140 | <div class="toolColorChipContainer"><div data-montage-id="fillColorCtrl" class="toolColorChipCtrl"></div></div> | 141 | <div data-montage-id="fillColorCtrlContainer" class="toolColorChipContainer"><div data-montage-id="fillColorCtrl" class="toolColorChipCtrl"></div></div> |
141 | <div class="nj-divider divider-vertical"> </div> | 142 | <div class="nj-divider divider-vertical"> </div> |
142 | <label class="label">Stroke:</label> | 143 | <label class="label">Stroke:</label> |
143 | <div data-montage-id="strokeControl" class="label"></div> | 144 | <div data-montage-id="strokeControl" class="label"></div> |
diff --git a/js/components/tools-properties/shape-properties.reel/shape-properties.js b/js/components/tools-properties/shape-properties.reel/shape-properties.js index 5829634c..7306275a 100755 --- a/js/components/tools-properties/shape-properties.reel/shape-properties.js +++ b/js/components/tools-properties/shape-properties.reel/shape-properties.js | |||
@@ -179,14 +179,14 @@ var ShapeProperties = exports.ShapeProperties = Montage.create(ToolProperties, { | |||
179 | this[this._selectedSubTool.properties].visible = true; | 179 | this[this._selectedSubTool.properties].visible = true; |
180 | 180 | ||
181 | if (this._selectedSubTool.id === "LineTool") { | 181 | if (this._selectedSubTool.id === "LineTool") { |
182 | this._fillColorCtrl.style["display"] = "none"; | 182 | this._fillColorCtrlContainer.style["display"] = "none"; |
183 | this._fillColorCtrl.visible = false; | 183 | this._fillColorCtrlContainer.visible = false; |
184 | this._fillColorCtrlIcon.style["display"] = "none"; | 184 | this._fillColorCtrlIcon.style["display"] = "none"; |
185 | this._fillColorCtrlIcon.visible = false; | 185 | this._fillColorCtrlIcon.visible = false; |
186 | this.endDivider.style["display"] = "none"; | 186 | this.endDivider.style["display"] = "none"; |
187 | } else { | 187 | } else { |
188 | this._fillColorCtrl.style["display"] = ""; | 188 | this._fillColorCtrlContainer.style["display"] = ""; |
189 | this._fillColorCtrl.visible = true; | 189 | this._fillColorCtrlContainer.visible = true; |
190 | this._fillColorCtrlIcon.style["display"] = ""; | 190 | this._fillColorCtrlIcon.style["display"] = ""; |
191 | this._fillColorCtrlIcon.visible = true; | 191 | this._fillColorCtrlIcon.visible = true; |
192 | this.endDivider.style["display"] = ""; | 192 | this.endDivider.style["display"] = ""; |