Fix editor search iteration

[jasonLaster]

Associated Issue: #1928

Summary of Changes

• add a test
• take modal out of the dom on close

[codecov]

Codecov Report

Merging #3546 into master will decrease coverage by <.01%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master    #3546      +/-   ##
==========================================
- Coverage   51.86%   51.86%   -0.01%     
==========================================
  Files         119      119              
  Lines        4647     4649       +2     
  Branches      954      955       +1     
==========================================
+ Hits         2410     2411       +1     
- Misses       2237     2238       +1

Impacted Files	Coverage Δ
src/components/SymbolModal.js	30.23% <50%> (+0.31%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update daebcfd...424d404. Read the comment docs.