Omniprompt broken if search is too long #130
Comments
Please raise the PR when you are ready with it. |
|
lol, i don't know how to fix it, i was just speculating. |
|
Please refer to this comment of mine. |
|
Refer to this link: https://stackoverflow.com/questions/30239092/how-to-get-multiline-input-from-user and if you can fix it, raise the PR. |
|
Or you can also use a wider terminal. |
|
O_o I think there's some cultural misunderstanding here. Leave the issues open, only close the ones that really need closing. Just slow down development. And get some helpers. (for the issue here, it's not about multiline input, the line doesn't wrap around properly, it should be like the python prompt) |
|
Few people care about open defects in open source projects. Check out the contribution graph. So now I see it as - if you need it, fix it yourself or use the available workaround. |
|
The api that I know of is input() and input() has this limitation. That needs to be addressed in python core/an alternative has to be provided instead of trying hacks in the caller's side. |
|
I see the problem is using color codes in diff --git a/ddgr b/ddgr
index f098cc5..1ffa036 100755
--- a/ddgr
+++ b/ddgr
@@ -1376,7 +1376,8 @@ class DdgCmd:
colors = self.colors
message = 'ddgr (? for help)'
- self.prompt = ((colors.prompt + message + colors.reset + ' ')
+ #self.prompt = ((colors.prompt + message + colors.reset + ' ')
+ self.prompt = (('\x1b[40m' + message + '\x1b[0m' + ' ')
if (colors and os.getenv('DISABLE_PROMPT_COLOR') is None) else (message + ': '))
@propertyYou can |
|
This workaround is fine. I have the impression, you should be using this library instead. You should let relevant issues open. You make it harder for people that care to contribute. |
|
I don't link to libraries for a single function call. |
If the search is too long, the UI gets bugged up in various ways. This is apparent, when you are crammed and lack space. The underlying logic works, it's just the displaying of the text of the search that is bugged.
from terminal, do a very long search. It does the search, with the arrows, get the previous search, then if you try to move the cursor to the beginning, you can't go all the way.
In the UI, if you type too much, it comes back and overwrights the same line. If you try backspace the line is no longer visible or in part. Again, can't move the cursor all the way to the start.
I think the solution, is to simply support multiple lines in the UI. The bugs is just the one line overflowing and doing unexpected stuff. (probably the same issue in googler, didn't test that)
The text was updated successfully, but these errors were encountered: