fix(ios): properly dismiss search controller after editing

[garymathews]

• Properly dismiss search controller after editing
  • Only one search controller can be active at a time

NOTE: No idea why the delay is necessary, it's originally from here

TEST CASE

1. Enter text into Search A, table should update with filtered results
2. Click Toggle List
3. Enter text into Search B, table should update with filtered results

const win = Titanium.UI.createWindow({ layout: 'vertical' });
const toggle = Ti.UI.createButton({ top: 80, title: 'Toggle List' })
const search_a = Ti.UI.createSearchBar({ hintText: 'Search A' });
const search_b = Ti.UI.createSearchBar({ hintText: 'Search B' });
const tableView_a = Ti.UI.createTableView({
    data: [ createSection(5, 'A') ],
    search: search_a,
    height: '40%'
});
const tableView_b = Ti.UI.createTableView({
    data: [ createSection(5, 'B') ],
    search: search_b,
    height: '40%',
    visible: false
});

toggle.addEventListener('click', e => {
    tableView_a.visible = tableView_b.visible;
    tableView_b.visible = !tableView_b.visible;

    if (!tableView_a.visible) {
        search_a.blur();
    }
    if (!tableView_b.visible) {
        search_b.blur();
    }
});

win.add([toggle, tableView_a, tableView_b]);
win.open();

function createSection(rows, suffix) {
    const section = Ti.UI.createTableViewSection({
        headerTitle: `Section ${suffix}`
    });

    for (let i = 1; i <= rows; i++) {
        section.add(Ti.UI.createTableViewRow({
            title: `Row #${i}`
        }));
    }

    return section;
}

JIRA Ticket

[build]

	Fails
🚫	Tests have failed, see below for more information.

	Messages
📖	✊ The commits in this PR match our conventions! Feel free to Rebase and Merge this PR when ready.
📖	❌ 2 tests have failed There are 2 tests failing and 726 skipped out of 10653 total tests.

Tests:

Classname	Name	Time	Error
android.emulator.Titanium.Network.HTTPClient	TLSv3 support (11)	0.21	Error: Could not determine TLSv3 support. at HTTPClient.onerror (/ti.network.httpclient.test.js:758:20)
ios.ipad.Titanium.UI.WebView	#findString() #findString with configuration (14.4.0)	30	Error: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (app.js)

Generated by 🚫 dangerJS against da0e359

[jquick-axway]

LGTM

I also tested it with my own ListView/TableView search bar test scripts and it works fine.

[ewanharris]

FR Pass!