binsync icon indicating copy to clipboard operation
binsync copied to clipboard

Started "improving" the Globals panel

Open themaks opened this issue 1 year ago • 2 comments

  • "Name" and "Address" are now 2 separate columns, for readability
  • Double clicking on a row jumps to the typedef/enum/struct definition, or on the variable's address

Depends on https://github.com/binsync/libbs/pull/125

DRAFT, still under tests

themaks avatar Oct 08 '24 13:10 themaks

Hey @themaks is this PR ready for review? Thanks so much for your contributions so far. They've been great :)

mahaloz avatar Oct 18 '24 15:10 mahaloz

Yes, this seems to work well, I ll let you test! Just a quick thing that is still buggy: the search function does not work as intended (if I remember correctly, it does not search in the Name column)

themaks avatar Oct 20 '24 08:10 themaks

Hey @themaks, sorry for the delay in the review. I'd love to merge this, but you were right, the search feature no longer works. Previously, you could search by:

  • name
  • committer

But now it seems you can only search by Type V, E, T, ... This is a big enough breaking change when dealing with large binaries. Any chance you could fix this? I think it's just a bug with what is the text content of the row.

mahaloz avatar Nov 04 '24 22:11 mahaloz

Thanks for looking into it, I'll check this morning!

sorry for the delay

This is an open source project you're gracefully giving time to, no need to feel sorry about anything 😁

themaks avatar Nov 05 '24 07:11 themaks

This is fixed, all types of searches work now:

  • by type (T, S, E, V)
  • by name
  • by address (lowercase hex, as displayed in the table)
  • by author

Cheers

themaks avatar Nov 05 '24 09:11 themaks

Nicely done! Everything works great now!

mahaloz avatar Nov 05 '24 16:11 mahaloz