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

Make it easier to modify charge quantities #314

Open
reactormonk opened this issue Jul 7, 2015 · 5 comments
Open

Make it easier to modify charge quantities #314

reactormonk opened this issue Jul 7, 2015 · 5 comments

Comments

@reactormonk
Copy link
Contributor

How I would do it: add an additional column with the number, and make that column editable.

@blitzmann
Copy link
Collaborator

I was actually just looking at this while working on #83. I do not think it is possible, or at least it is not straightforward, to do this. But I haven't taken more than a glance of it. I will look into it more thoroughly when I've got time and report back. =)

@reactormonk
Copy link
Contributor Author

I've only messed with tk, and there it's pretty simple. Let's see what wx is like.

@reactormonk
Copy link
Contributor Author

Apparently it's possible, but you'll have to use a grid instead of a list. http://docs.wxwidgets.org/trunk/classwx_grid_cell_editor.html

@blitzmann
Copy link
Collaborator

Yeah, with the preliminary search last night it seems the plain old ListCtrl is not able - we'd have to switch to UltimateListCtrl or a Grid. But that's the thing. To switch over from ListCtrl... I'm not sure how involved that is. Note that all of out "lists" (fitting window, cargo, drones, market, projected items, etc) all utilize our base Display class which is based off of ListCtrl. To be an easy switch, they would have to have similar APIs (Ultimate may be better in this regard). Whether this is worth the effort I'm not sure.

I'll continue looking into this in my spare time. In the meantime, feel free to poke the source and if you want to undertake development on something like this, go for it. Even if you only get a rough proof of concept. If you have any questions about the source, let me know =)

@blitzmann
Copy link
Collaborator

Was going through the wxpython demos for another reason, and ran accross the edit mixin for list ctrl.

http://www.wxpython.org/docs/api/wx.lib.mixins.listctrl.TextEditMixin-class.html

Apparently this allows inline editing of the ctrl. Dunno how practical it would be to apply this or if it would work out well, but it's there to look into when I've got time.

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

2 participants