Skip to content

Conversation

@taooceros
Copy link
Member

resolves #701

@jjw24 jjw24 added enhancement New feature or request bug Something isn't working and removed enhancement New feature or request labels Sep 26, 2021
@jjw24 jjw24 added this to the 1.9.0 milestone Sep 26, 2021
@jjw24
Copy link
Member

jjw24 commented Sep 26, 2021

@taooceros have you tested this sovles the issue 701?

I remember the requery doesnt work from the context menu, do you know if this also addresses it?

@jjw24 jjw24 enabled auto-merge September 26, 2021 23:01
@taooceros
Copy link
Member Author

@taooceros have you tested this sovles the issue 701?

I remember the requery doesnt work from the context menu, do you know if this also addresses it?

Hmm I don't think so

{
QueryText = queryText;
}
else if (reQuery)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like this shouldnt be an 'else if', they should be two seperate statements

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well actually, when QueryText is not equal to queryText, the requery will be triggered. Only when they are the same, we need manually requery.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ohhh it's requeried in the setter method...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lets add a comment in to make it clear

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i will commit

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I remember it is the optimization from fody xd

@jjw24 jjw24 added the review in progress Indicates that a review is in progress for this PR label Sep 27, 2021
@taooceros
Copy link
Member Author

the vscode online makes these kinds of quick commit much easier lol

@jjw24 jjw24 removed the review in progress Indicates that a review is in progress for this PR label Sep 27, 2021
@jjw24 jjw24 merged commit 25b4e2b into dev Sep 27, 2021
@jjw24 jjw24 deleted the PluginHotkeyRequery branch September 27, 2021 04:06
@jjw24
Copy link
Member

jjw24 commented Sep 27, 2021

i keep forgetting to use vscode online lol

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

None yet

Development

Successfully merging this pull request may close these issues.

results not refreshing if keyword is the same as last time

3 participants