Discource-C
/
discourse
mirror of https://github.com/discourse/discourse.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

UX: ignore leading/trailing whitespaces in site settings search

This commit is contained in:
Régis Hanol 2019-05-15 22:22:20 +02:00
parent b9fcb2bcdf
commit db89722a98
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/assets/javascripts/admin/controllers/admin-site-settings.js.es6
View File

@ -12,7 +12,7 @@ export default Ember.Controller.extend({
let filter; let filter;
if (this.get("filter")) { if (this.get("filter")) {
filter = this.get("filter").toLowerCase(); filter = this.get("filter").toLowerCase().trim();
} }
if ((!filter || 0 === filter.length) && !this.get("onlyOverridden")) { if ((!filter || 0 === filter.length) && !this.get("onlyOverridden")) {

2
test/javascripts/acceptance/admin-site-settings-test.js.es6
View File

@ -42,7 +42,7 @@ QUnit.test("upload site setting", async assert => {
QUnit.test("changing value updates dirty state", async assert => { QUnit.test("changing value updates dirty state", async assert => {
await visit("/admin/site_settings"); await visit("/admin/site_settings");
await fillIn("#setting-filter", "title"); await fillIn("#setting-filter", " title ");
assert.equal(count(".row.setting"), 1, "filter returns 1 site setting"); assert.equal(count(".row.setting"), 1, "filter returns 1 site setting");
assert.ok(!exists(".row.setting.overridden"), "setting isn't overriden"); assert.ok(!exists(".row.setting.overridden"), "setting isn't overriden");