jglim / CaesarSuite

Library and applications to work with Dаіmlеr diagnostics CBF files.
MIT License
125 stars 33 forks source link

contextMenuLimit in VCForm #3

Closed N0cynym closed 3 years ago

N0cynym commented 3 years ago

Is it possible to offer a option to set this limit? For example vgsnag2 provides a lot of possible VC and to verify the VC function it whould be nice to test, program and verfiy the string on test bench with vgnag2 device and Monaco/Vediamo.

Or should I compile it local with higher limit?

jglim commented 3 years ago

Hi N0cynym,

I took a look at vgsnag2, and indeed there is too many entries to reasonably show in a context menu. I've added a selection dialog (with scrollbars) which will not be constrained by the context menu limit. Thanks for pointing this out.

image

If this is still insufficient, let me know and I will reopen the issue and try to find a different solution.

N0cynym commented 3 years ago

Is it possible to add txtFilter for search in Form?

jglim commented 3 years ago

Sure - added in v1.3.2 https://github.com/jglim/CaesarSuite/releases/tag/v1.3.2

N0cynym commented 3 years ago

Can we set this higher? I mean you can find the segments with the search but I think it's better to show them all.

jglim commented 3 years ago

I might have to add a separate settings page that also includes a configurable limit (individual preference: the current limit is quite comfortable for me). This is definitely doable, but might take a while as I would want to aggregate a bunch of user settings before adding a preferences page.

N0cynym commented 3 years ago

Definitely a better way! Thanks for informing about this plan.