diff options
author | suelockwood <deathbear@apache.org> | 2013-12-19 10:38:25 -0500 |
---|---|---|
committer | suelockwood <deathbear@apache.org> | 2013-12-19 10:38:25 -0500 |
commit | 299657473f0f0aea3314183f87fe7921bdd29c75 (patch) | |
tree | bdb5f7b06b28c95bf6647d77c9914ed1f5284c98 | |
parent | 35968da547dc098c257c0de14e08dfeb1768a3b0 (diff) | |
download | couchdb-299657473f0f0aea3314183f87fe7921bdd29c75.tar.gz |
Fix for issue: Search for Document with name that doesn't exist leads to unworkable state
Turned it into a form. Added validation
-rw-r--r-- | src/fauxton/app/modules/documents/views.js | 14 | ||||
-rw-r--r-- | src/fauxton/app/templates/documents/jumpdoc.html | 8 | ||||
-rw-r--r-- | src/fauxton/assets/less/fauxton.less | 15 |
3 files changed, 18 insertions, 19 deletions
diff --git a/src/fauxton/app/modules/documents/views.js b/src/fauxton/app/modules/documents/views.js index faf3539b6..44cf48468 100644 --- a/src/fauxton/app/modules/documents/views.js +++ b/src/fauxton/app/modules/documents/views.js @@ -1602,16 +1602,22 @@ function(app, FauxtonAPI, Components, Documents, Databases, pouchdb, resizeColum }, events: { - "submit #jump-to-doc": "jumpToDoc", - "click #jump-to-doc-label": "jumpToDoc" + "submit #jump-to-doc": "jumpToDoc" }, jumpToDoc: function (event) { event.preventDefault(); - var docId = this.$('#jump-to-doc-id').val(); + var docId = this.$('#jump-to-doc-id').val().trim(); - FauxtonAPI.navigate('/database/' + this.database.id +'/' + docId, {trigger: true}); + if (this.database.allDocs.where({"_id":docId}).length > 0){ + FauxtonAPI.navigate('/database/' + this.database.id +'/' + docId, {trigger: true}); + } else { + FauxtonAPI.addNotification({ + msg: 'Document ID does not exist.', + type: 'error' + }); + } }, afterRender: function () { diff --git a/src/fauxton/app/templates/documents/jumpdoc.html b/src/fauxton/app/templates/documents/jumpdoc.html index c6f46523b..43fdb9c3a 100644 --- a/src/fauxton/app/templates/documents/jumpdoc.html +++ b/src/fauxton/app/templates/documents/jumpdoc.html @@ -12,8 +12,8 @@ License for the specific language governing permissions and limitations under the License. --> -<form id="jump-to-doc" class="form-inline" > - <label id="jump-to-doc-label" class="fonticon-search"> - <input type="text" id="jump-to-doc-id" class="input-large" placeholder="Document ID"></input> - </label> +<form id="jump-to-doc" class="form-inline input-append" > + <input type="text" id="jump-to-doc-id" class="input-large" placeholder="Document ID"></input> + + <button class="fonticon-search btn button red " type="submit"></button> </form> diff --git a/src/fauxton/assets/less/fauxton.less b/src/fauxton/assets/less/fauxton.less index b94ba39ae..0cf7bc2b9 100644 --- a/src/fauxton/assets/less/fauxton.less +++ b/src/fauxton/assets/less/fauxton.less @@ -975,18 +975,11 @@ div.spinner { } #jump-to-doc { - width: 50%; - max-width: 600px; + width: auto; float:right; - margin-right: 40px; - - #jump-to-doc-label { - width: 100%; - } - - #jump-to-doc-id { - width: 100%; - margin-top: -4px; + button{ + padding-left: 20px; + padding-right: 10px; } } |