Timeline for AutoSuggest: Better way to implement focus after inserting a snippet
Current License: CC BY-SA 4.0
14 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| S Apr 9, 2019 at 11:01 | history | bounty ended | CommunityBot | ||
| S Apr 9, 2019 at 11:01 | history | notice removed | CommunityBot | ||
| Apr 8, 2019 at 17:34 | history | edited | Sᴀᴍ Onᴇᴌᴀ♦ |
edited tags
|
|
| Apr 8, 2019 at 5:40 | history | edited | Sᴀᴍ Onᴇᴌᴀ♦ |
edited tags
|
|
| Apr 8, 2019 at 5:40 | answer | added | Sᴀᴍ Onᴇᴌᴀ♦ | timeline score: 2 | |
| Apr 5, 2019 at 5:43 | comment | added | AvcS | I have posted both the snippets I want reviewed and Also if you can suggest better names for those params, please do. I am also little confused about how to name them :D | |
| Apr 5, 2019 at 5:37 | comment | added | AvcS |
I have included the logic for focusText in the question, the variables are a little different in original repo, you can find it in setValue method, line number 214
|
|
| Apr 3, 2019 at 19:45 | comment | added | Sᴀᴍ Onᴇᴌᴀ♦ |
You stated: "it simplifies the logic for focusText" that makes it sound like focusText is a function/method but after searching the repository that doesn't appear to be the case... what logic is simplified by having negative numbers? Please edit your post to include any logic you want reviewed...
|
|
| Apr 1, 2019 at 15:00 | history | tweeted | twitter.com/StackCodeReview/status/1112731467287793669 | ||
| S Apr 1, 2019 at 9:14 | history | bounty started | AvcS | ||
| S Apr 1, 2019 at 9:14 | history | notice added | AvcS | Draw attention | |
| Mar 25, 2019 at 10:12 | history | edited | AvcS | CC BY-SA 4.0 |
deleted 6 characters in body
|
| Mar 25, 2019 at 6:10 | review | First posts | |||
| Mar 25, 2019 at 7:25 | |||||
| Mar 25, 2019 at 6:09 | history | asked | AvcS | CC BY-SA 4.0 |