Skip to content

Conversation

Garulf
Copy link
Member

@Garulf Garulf commented Dec 19, 2021

Adds a slight offset to the comparison of Rendered text and text box length.

Before:
image

After:
image

@Garulf Garulf self-assigned this Dec 19, 2021
@Garulf Garulf added the bug Something isn't working label Dec 19, 2021
@Garulf Garulf changed the title Add offset Add offset to suggestion text hide calculation Dec 19, 2021
@Garulf Garulf added this to the 1.9.1 milestone Dec 19, 2021
@jjw24
Copy link
Member

jjw24 commented Dec 19, 2021

can you send me a string to test, i cant replicate what you got in the screenshot.

@Garulf
Copy link
Member Author

Garulf commented Dec 20, 2021

can you send me a string to test, i cant replicate what you got in the screenshot.

It's tricky because the text has to be just at the cut off after you tab complete it.

any chance we can not hard code 40 but get it dynamically?

Offset is adjustable in settings file. Not sure if it's really needed in GUI.

EDIT: Which I'm now realizing is not what you meant... I would need to some how grab the right-padding being set by the theme on the QueryTextbox

@jjw24
Copy link
Member

jjw24 commented Dec 20, 2021

Able to replicate with current release 1.9.0:
Auto complete for websearch enabled
search guys only
auto tab to guys only want one thing
then auto tab to guys only want one thing and it's disgusting tweet

image

@jjw24 jjw24 enabled auto-merge December 20, 2021 11:00
@jjw24 jjw24 merged commit 47c4023 into Flow-Launcher:dev Dec 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants