Giter VIP home page Giter VIP logo

Comments (6)

paxcut avatar paxcut commented on June 20, 2024

this was fixed two or three days ago. now the popup can be moved and it also becomes transparent when not hovering. the amount of transparency can be adjusted in the theme editor. Note that the problem was that the window was already scrolled all the way to the top, so you couldn't scroll down any further to make the content visible because when the search popup is open the mouse wheel scrolls the window up and down.

Please obtain the latest nightly build that already incorporates said changes so you can verify this is no longer a problem for you.

image

from imhex.

mehlian avatar mehlian commented on June 20, 2024

I'll wait for next release since nightly build (available from readme page button) is in msi format and I want portable.

Note that the problem was that the window was already scrolled all the way to the top, so you couldn't scroll down any further to make the content visible because when the search popup is open the mouse wheel scrolls the window up and down.

I don't understand this statement. It is true that for small binaries where there is no scroll bar there is no way to scroll view, but I can't scroll even when pattern was found in the middle of big file and search panel was still visible (search panel was in focus but mouse was over Hex editor to scroll Hex editor view).

I would prefer search panel as bar at the bottom than moving search panel left and right to see what was found and where.

Search panel in version 1.33.X@8802a8e can have more issues like you cannot resize ImHex window while search panel is visible.

from imhex.

paxcut avatar paxcut commented on June 20, 2024

Head on over to this page and at the very bottom you'll find portable nightly builds.

but I can't scroll even when pattern was found in the middle of big file and search panel was still visible (search panel was in focus but mouse was over Hex editor to scroll Hex editor view).

The search window in the hex editor is implemented as an ImGui Modal window that needs focus while is open and closes once focus is lost, so it is impossible to resize other windows or anything that requires focus. Scrolling the hex editor view does not require focus to the hex editor itself. As long as the search panel has the focus it needs to stay open, the mouse scroll wheel will scroll the hex editor window up and down. It is easy to test, open the search window and move the mouse wheel up and down.

I would prefer search panel as bar at the bottom than moving search panel left and right to see what was found and where.

Please download a portable nightly build from the web page listed above to test the new functionality and comment on it if there are further issues.

Search panel in version 1.33.X@8802a8e can have more issues like you cannot resize ImHex window while search panel is visible.

As explained earlier that's an implementation limitation that cannot be helped. Unlike the pattern editor search window that can make room to avoid hiding any content, like visual studio code does, the hex editor widow has no room to allow placing the search panel in a fixed position that is guaranteed not to hide some content which is why it was decided to allow moving it and giving it transparency so that no data is permanently hidden under it.

from imhex.

mehlian avatar mehlian commented on June 20, 2024

Downloaded nightly build and here are results:

  • scroll 'Hex editor' view while 'search panel' is visible - fixed
  • resize ImHex window while 'search panel' is visible - fixed

I have found new issue with 'search panel' logic. I'll open new issue. This can be closed on next release.

from imhex.

mehlian avatar mehlian commented on June 20, 2024

Unlike the pattern editor search window that can make room to avoid hiding any content, like visual studio code does, the hex editor widow has no room to allow placing the search panel in a fixed position that is guaranteed not to hide some content which is why it was decided to allow moving it and giving it transparency so that no data is permanently hidden under it.

In visual studio code when we use ctrl+f to show search panel in code window, extra padding/margin is added at top to allow scrolling code down and look at code covered by search panel.

It may be 'ugly' behavior for 'hex editor' view.

That's why search bar at the bottom would be better - content of 'hex editor' view would not move, just new search bar would appear at bottom covering bottom portion of 'hex editor'. Scrollbar would move up (above search bar) and you could scroll down to uncover this area.

Similarly how ctrl+f works in Firefox browser.

from imhex.

paxcut avatar paxcut commented on June 20, 2024
  • scroll 'Hex editor' view while 'search panel' is visible - fixed
  • resize ImHex window while 'search panel' is visible - fixed

Glad to hear it and you don't need to wait for release to close the issue if you don't want to.

In visual studio code when we use ctrl+f to show search panel in code window, extra padding/margin is added at top to allow scrolling code down and look at code covered by search panel.
It may be 'ugly' behavior for 'hex editor' view.

Yes, I mentioned that because I coded the Pattern editor search window to behave like vs code and ensuring that the text doesn't jump up or down when you hit ctrl-f wasn't easy (in fact there are cases when closing the search panel where the text has to jump as vs code shows). The search panel for the text editor is designed wide and span only two or three (with replace) lines of code so the extra padding needed is minimal and the end of scroll cases, where the main problem lies really, can be handled gracefully.

That kind of behavior is just not possible with the hex editor window because in general there is no region of the layout that's not covering some portion of the ui and the window spans too many lines to trick the eye into thinking that nothing has changed unless you are paying close attention. Think NotePad++ search window which has a similar issue and look at how it is handled there. It may not look it, but there is a great deal of planning and study of other app uis that goes into the design choices being made. You can't ignore some needs to justify some choice without some sound logic to back it up.

That's why search bar at the bottom would be better - content of 'hex editor' view would not move, just new search bar would appear at bottom covering bottom portion of 'hex editor'. Scrollbar would move up (above search bar) and you could scroll down to uncover this area.

Similarly how ctrl+f works in Firefox browser

As an example, since you mentioned it, turning things into a firefox-like search is not an option for Imhex because the hex editor search window is no longer a simple text search window and must allow for the search of binary values and other things that turns that into an unfeasible choice.
There are lengthy discussions on Imhex discord about this and other ui considerations and everyone is welcome to participate and express their opinion but the final choices can only be made by the main developers. That's a tall order since the very popularity of the app depends on how well tuned the choices are to the community need and I know of few open source projects that are more responsive to the community than Imhex is. But I am digressing so I'll stop here.

from imhex.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.