You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@couchdb.apache.org by de...@apache.org on 2014/02/27 22:52:27 UTC
couchdb commit: updated refs/heads/2127-config-edit-row to 54312fa
Repository: couchdb
Updated Branches:
refs/heads/2127-config-edit-row [created] 54312fa2a
Fixed the configs to be double click to edit.
Fixed the templates to not use IDs as selectors (BAD)
Fixed the underscore values for inputs to be HTML-escaped
Project: http://git-wip-us.apache.org/repos/asf/couchdb/repo
Commit: http://git-wip-us.apache.org/repos/asf/couchdb/commit/54312fa2
Tree: http://git-wip-us.apache.org/repos/asf/couchdb/tree/54312fa2
Diff: http://git-wip-us.apache.org/repos/asf/couchdb/diff/54312fa2
Branch: refs/heads/2127-config-edit-row
Commit: 54312fa2ad1d31e6af603ef4a90a7560bf69bc9f
Parents: 055219f
Author: suelockwood <de...@apache.org>
Authored: Thu Feb 27 16:52:16 2014 -0500
Committer: suelockwood <de...@apache.org>
Committed: Thu Feb 27 16:52:16 2014 -0500
----------------------------------------------------------------------
.../app/addons/config/assets/less/config.less | 16 +++++++---------
src/fauxton/app/addons/config/resources.js | 18 +++++++++---------
.../app/addons/config/templates/dashboard.html | 4 ++--
src/fauxton/app/addons/config/templates/item.html | 18 +++++++++---------
4 files changed, 27 insertions(+), 29 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/couchdb/blob/54312fa2/src/fauxton/app/addons/config/assets/less/config.less
----------------------------------------------------------------------
diff --git a/src/fauxton/app/addons/config/assets/less/config.less b/src/fauxton/app/addons/config/assets/less/config.less
index 88bbc66..7d8da6a 100644
--- a/src/fauxton/app/addons/config/assets/less/config.less
+++ b/src/fauxton/app/addons/config/assets/less/config.less
@@ -11,19 +11,17 @@
* the License.
*/
.config-item {
- height: 41px;
+ height: 65px;
- td:hover .edit-button {
- display: block;
+ .js-value-input {
+ width: 80%;
+ margin: 0;
}
-
- .value-input {
- width: 98%;
- }
-
- #delete-value {
+ .js-show-value,
+ .js-delete-value {
cursor: pointer;
}
+ button {width: 7%;}
}
#add-section-modal {
http://git-wip-us.apache.org/repos/asf/couchdb/blob/54312fa2/src/fauxton/app/addons/config/resources.js
----------------------------------------------------------------------
diff --git a/src/fauxton/app/addons/config/resources.js b/src/fauxton/app/addons/config/resources.js
index 14d2474..445fc01 100644
--- a/src/fauxton/app/addons/config/resources.js
+++ b/src/fauxton/app/addons/config/resources.js
@@ -76,10 +76,10 @@ function (app, FauxtonAPI) {
template: "addons/config/templates/item",
events: {
- "click .edit-button": "editValue",
- "click #delete-value": "deleteValue",
- "click #cancel-value": "cancelEdit",
- "click #save-value": "saveValue"
+ "dblclick .js-edit-value": "editValue",
+ "click .js-delete-value": "deleteValue",
+ "click .js-cancel-value": "cancelEdit",
+ "click .js-save-value": "saveValue"
},
deleteValue: function (event) {
@@ -92,18 +92,18 @@ function (app, FauxtonAPI) {
},
editValue: function (event) {
- this.$("#show-value").hide();
- this.$("#edit-value-form").show();
+ this.$(".js-show-value").hide();
+ this.$(".js-edit-value-form").show();
},
saveValue: function (event) {
- this.model.save({value: this.$(".value-input").val()});
+ this.model.save({value: this.$(".js-value-input").val()});
this.render();
},
cancelEdit: function (event) {
- this.$("#edit-value-form").hide();
- this.$("#show-value").show();
+ this.$(".js-edit-value-form").hide();
+ this.$(".js-show-value").show();
},
serialize: function () {
http://git-wip-us.apache.org/repos/asf/couchdb/blob/54312fa2/src/fauxton/app/addons/config/templates/dashboard.html
----------------------------------------------------------------------
diff --git a/src/fauxton/app/addons/config/templates/dashboard.html b/src/fauxton/app/addons/config/templates/dashboard.html
index b7dbc55..c14402c 100644
--- a/src/fauxton/app/addons/config/templates/dashboard.html
+++ b/src/fauxton/app/addons/config/templates/dashboard.html
@@ -20,8 +20,8 @@ the License.
</div>
<table class="config table table-striped table-bordered">
<thead>
- <th id="config-section"> Section </th>
- <th id="config-option"> Option </th>
+ <th id="config-section" width="20%"> Section </th>
+ <th id="config-option" width="20%"> Option </th>
<th id="config-value"> Value </th>
<th id="config-trash"></th>
</thead>
http://git-wip-us.apache.org/repos/asf/couchdb/blob/54312fa2/src/fauxton/app/addons/config/templates/item.html
----------------------------------------------------------------------
diff --git a/src/fauxton/app/addons/config/templates/item.html b/src/fauxton/app/addons/config/templates/item.html
index 502c9b3..a628fe6 100644
--- a/src/fauxton/app/addons/config/templates/item.html
+++ b/src/fauxton/app/addons/config/templates/item.html
@@ -17,15 +17,15 @@ the License.
<% } else { %>
<td></td>
<% } %>
-<td> <%= option.name %> </td>
-<td>
- <div id="show-value">
- <%= option.value %> <button class="btn btn-mini pull-right hide edit-button"> Edit </button>
+<td > <%= option.name %> </td>
+<td class="js-edit-value">
+ <div class="js-show-value">
+ <%= option.value %>
</div>
- <div id="edit-value-form" style="display:none">
- <input class="value-input" type="text" value="<%= option.value %>" />
- <button id="save-value" class="btn btn-success btn-small"> Save </button>
- <button id="cancel-value" class="btn btn-small"> Cancel </button>
+ <div class="js-edit-value-form" style="display:none">
+ <input class="js-value-input" type="text" value="<%- option.value %>" />
+ <button class="js-save-value btn btn-success fonticon-circle-check btn-small"> </button>
+ <button class="js-cancel-value btn btn-small fonticon-circle-x"> </button>
</div>
</td>
-<td id="delete-value" class="text-center"> <i class="icon-trash"> </i> </td>
+<td class="js-delete-value" class="text-center"> <i class="icon-trash"> </i> </td>