Skip to content
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

Scrollback Search activates Transparency on Lost Focus #497

Closed
ElvenSpellmaker opened this issue Nov 25, 2015 · 6 comments
Closed

Scrollback Search activates Transparency on Lost Focus #497

ElvenSpellmaker opened this issue Nov 25, 2015 · 6 comments

Comments

@ElvenSpellmaker
Copy link

The Alt+F3 scrollback search activates the transparency as if the window has become unfocussed. This makes it hard to look at the content while searching.

Pretty sure the window should count as focussed still.

@mintty
Copy link
Owner

mintty commented Nov 25, 2015

@kiwiz: Might you have any idea?

@kiwiz
Copy link

kiwiz commented Nov 26, 2015

Since we have two windows now, we can't listen for the WM_SETFOCUS event. This gets fired whenever the main console pane loses focus to the search bar.

Instead, we can watch for the WM_ACTIVATE message to determine whether the window is active. I've made the necessary changes in the diff below. However, the caret will act oddly if you have your cursor set to blinking. This is probably because the screen is getting redrawn on every cursor blink, causing the search caret to disappear.

To resolve this, I think some refactoring of the code to not use has_focus will be necessary. However, before I attempt that, I want to see if anyone has a better idea.

tmp.txt

@mintty
Copy link
Owner

mintty commented Nov 26, 2015

Thanks a lot for the quick semi-fix. I have no cursor in the search bar, though, regardless of blinking.

@mintty
Copy link
Owner

mintty commented Nov 27, 2015

After some tweaking of focus handling, uploaded fix. Please check.

@kiwiz
Copy link

kiwiz commented Nov 27, 2015

Looks good to me! I was afraid this would require a lot more refactoring, but this works just fine. 👍

@mintty
Copy link
Owner

mintty commented Feb 5, 2016

Released in 2.2.3.

@mintty mintty closed this as completed Feb 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants