-
Notifications
You must be signed in to change notification settings - Fork 90
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
Suggestions interface #55
Comments
@svineet can you, please, give me more information about the UI refinement? |
moved from PR: @shardulc @jonorthwash I went ahead and set it up on my server (with a recaptcha key) so we can easily test: https://azurite.skc.name/apertium-html-tools/build/index.eng.html You need to check There are definitely some UI touches that need to take place but I haven't thoroughly tested it yet. Whoever can get to it first should take a crack at it! I also have a hunch that it's making my CPU spike... Here be dragons. Would appreciate it if someone could confirm that it's not just me. |
@sushain97 When I click the suggest button, nothing happens and I cannot see any network activity either. (The captcha works fine.) However, I can't debug this because I only see |
Let me run make debug next time I'm not afk and it'll be easier.
…On Dec 4, 2017 12:33 PM, "Shardul Chiplunkar" ***@***.***> wrote:
@sushain97 <https://github.com/sushain97> When I click the suggest
button, nothing happens and I cannot see any network activity either. (The
captcha works fine.) However, I can't debug this because I only see min.js
😃
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#55 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEBEfq0y98seLmvkA5LpvPRm-gGus5fyks5s9CzNgaJpZM4IfB4S>
.
|
@shardulc https://azurite.skc.name/apertium-html-tools/build/index.debug.html Don't try too much to debug it ;) Just make a list of problems. |
List of problems:
|
Seems like a good set of issues to me, i.e. sufficiently specific to create a GCI task. Did you not experience CPU spikes while typing in the source text on this branch? |
I tried 10 times and I experienced a CPU spike only once, on my first try, on one of four cores. The other 9 times were completely normal CPU activity. |
The suggestions work on the backend was merged into the suggestions branch in this PR: apertium/apertium-apy#18
Overall, the suggestions route works and is almost production ready, but the following needs to be taken care of (feel free to add anything in the comments):
Current interface is at https://github.com/svineet/apertium-html-tools/tree/suggest-word and some of that work was merged into https://github.com/goavki/apertium-html-tools/tree/suggestions but my later improvements still have open PRs to the suggestions branch of html-tools. So we need to work on organizing that stuff.
The text was updated successfully, but these errors were encountered: