-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug] Search input cannot be reselected #22
Comments
Allow shift+enter to search backwards Don't search hidden rows/cols Add a search bar to allow searching a sheet https://github.com/audreyt/ethercalc/search?q=search&type=Commits&utf8=%E2%9C%93 |
Thanks @eddyparkinson I checked and all these are already committed in this SocialCalc repo (in fact, I copied over all the socialcalc files from ethercalc and face the same issue). I think I am going to remove the search feature for now until we understand better how input handling is done. |
Further, the only jQuery dependency is the search feature, I recommend it either becomes vanilla JS or gets removed. |
the GUI formulas, e.g. =SELECT("Female","-,Male,Female") |
I'll check to see if those formulas are in the socialcalc repo or just the ethercalc one |
they are in the socialcalc repo, afaik |
If we run a search, then click on a cell, we can't re-select the search input.
In fact, selected cells are offset vertically by one.
The search functionality looks like it was added (uses jQuery and different coding conventions). Seems it's in conflict with the keyboard event managers.
However, ethercalc does not have this problem (can't find the source in the ethercalc repo to compare).
Do you guys remember the change you made?
The text was updated successfully, but these errors were encountered: