rogershen / chrome-regex-search

Other
267 stars 56 forks source link

Highlight search text when re-opening the popup #4

Closed cxw42 closed 7 years ago

cxw42 commented 7 years ago

Roger, I got inspired :) . The substantive change is at the very end of src/js/popup.js . Resolves #3 . I also noticed that you were using two-space indents almost everywhere, and I changed the few remaining tabs to spaces for consistency.

Tested on Chrome 51.0.2704.103 (Official Build) m (32-bit) running on Windows 8.1. Works OK for me on both file: and https: URLs. Thanks!

Chris

cxw42 commented 7 years ago

Sorry for the confusion - replacing this with a separate PR for a branch just with these changes. I am working on some other goodies on cxw42:master :) .