Skip to content

Make "find and replace" window always appear on top of the parent editor #6603

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

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

facchinm
Copy link
Member

@facchinm facchinm commented Aug 10, 2017

Fixes #6408 and #4883

@arduino arduino deleted a comment from ArduinoBot Aug 10, 2017
@facchinm facchinm added this to the Next milestone Aug 18, 2017
@facchinm facchinm added the Component: IDE user interface The Arduino IDE's user interface label Aug 18, 2017
@facchinm facchinm added the Help wanted Arduino would especially appreciate assistance from the community on this item label Sep 7, 2017
@arduino arduino deleted a comment from ArduinoBot Sep 7, 2017
@czghzc
Copy link

czghzc commented Oct 9, 2017

2017-10-07_092746

I'm sorry, but now I understand your reply, it is ashamed.
I tested it, and now the find window is above all windows, but it finds another problem.
If there are other windows in the system, such as a browser, the find window will also be on top of the browser window, which will interfere with the operation of the browser window, and we hope that the find window is just above the Arduino main window.

What we want is like this picture below:
2017-10-07_093944

@facchinm
Copy link
Member Author

facchinm commented Oct 9, 2017

Hi @czghzc ,
I had the chance to test it on Windows and indeed it behaves exactly as you described.
I just pushed 09f5196 which should make the behaviour consistent across all OSes. The build by @ArduinoBot should arrive soon 😉

@arduino arduino deleted a comment from ArduinoBot Oct 9, 2017
@czghzc
Copy link

czghzc commented Oct 10, 2017

Hi @facchinm ,
I tested the windows version because i only had windows system, just as I wanted it, it worked very well and thank you so much thanks!

If the editor is focused, acquire alwaysOnTop capabilities but drop them as soon as both the editor and the dialog are out of focus
As implemented in Notepad++, a partially transparent search/replace window could make it easier to interact with the editor (particularly in fullscreen contexts, when the "always on top" window could hide the search results).
Unfortunately, Java can only apply the transparency to undecorated window, so it's a no-go.
Java gurus, please come to the rescue :D
Probably this approach could be adopted by all OSes with a little care on the window lifecycle.
@cmaglie cmaglie force-pushed the find_replace_window_top branch from 09f5196 to 71418ca Compare January 26, 2018 10:23
@kenneth558
Copy link

kenneth558 commented Mar 4, 2018

With 6603 the behavior is noticeably better, for doggone sure! (Linux Mint) Only thing is, still, I when I file-save via the menu dropdown, the "find" window still goes behind right when I select the final menu selection and stays there. I some other menu selections do the same (some in help menu, at least), but others (save-as, for example) work the "find" window just fine.

@czghzc
Copy link

czghzc commented Mar 9, 2018

pic2
The beginning is good, there is no problem, but after clicking the window's title bar with a mouse is not normal, the search window goes to the back of the main window.

When programming in C++, setting the search window to the child window of the main window can achieve the desired effect. However, I will not use Java. I do not know if Java has the same operation method.

In addition, I also have a suggestion that when the search window comes out, the content of the search text box is selected, which facilitates us to modify the content to be searched.

@kenneth558
Copy link

kenneth558 commented Mar 9, 2018

I agree with causing the text to be selected when re-pressing ctrl-f. I just thought you had a reason for not having it that way in the first place. Otherwise oftentimes a person enters new test unintentionally appending to previous. Additionally, you could replace the text with any text currently selected in the main window, just like when the search gets newly activated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: IDE user interface The Arduino IDE's user interface Help wanted Arduino would especially appreciate assistance from the community on this item
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants