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

Suggestion: Additional cell display options for List data. #63

Open
MvGulik opened this issue Mar 24, 2017 · 2 comments
Open

Suggestion: Additional cell display options for List data. #63

MvGulik opened this issue Mar 24, 2017 · 2 comments
Labels

Comments

@MvGulik
Copy link

@MvGulik MvGulik commented Mar 24, 2017

When a cell contains Group type data is has a number of displayed options to display its data.

I think it would be nice if something like that was also possible with List data.
It would, for example, give a bit more controle over the display width of the tables.

image

@jcupitt
Copy link
Member

@jcupitt jcupitt commented Mar 24, 2017

That's a good idea. It's not hard to implement, you need to define two classes, List (for example) and Listview, one handles the link to the programming language, one handles the display on the screen.

For example, group.c defines the Group class, about 200 lines of code, most of it to implement save-as on groups of images:

https://github.com/jcupitt/nip2/blob/master/src/group.c

And Valueview controls the display, about 120 lines of code:

https://github.com/jcupitt/nip2/blob/master/src/value.c

I think the problem is that nip2 is still a gtk2 program. It really needs updating to gtk3 (gtk4 is coming soon, argh) before committing much more work to it.

I've done a nip3 branch here:

https://github.com/jcupitt/nip2/blob/nip3

I've ported everything, it builds, but it doesn't run. All of the image display stuff has changed for gtk3, so it needs reimplementing.

I've done a vips gtk3 image viewer here:

https://github.com/jcupitt/vipsdisp

That builds and runs, and looks quite nice, but it's not really finished, it's missing a few features, and it has an annoying flicker when you zoom.

So the nip2 roadmap is currently:

  • finish gtk3 vipsdisp
  • copy-paste that into the nip3 branch, replacing the old nip2 image display code
  • a few more fixups to get a nip2 that basically works with gtk3

Then there are a load of things that need fixing in nip2:

  • move to vips8 API, get rid of the nip2 cache
  • rework the look n feel to fit the gtlk3 model

That would be enough to make something that could be released. The nip2 TODO has a long list of more blue-sky ideas.

Anyway, not too much can happen before the move to gtk3, unfortunately. I'm hoping that now vips8 is stabilising I'll have some time to fix up nip2.

@jcupitt jcupitt added the enhancement label Mar 24, 2017
@MvGulik
Copy link
Author

@MvGulik MvGulik commented Mar 25, 2017

I see.

As far as Nip2 goes, its working pretty nice. ie: no real showstoppers known at this side.
If the plan is to update nip2 to gtk3, I would give that more precedents over smoothing out nip2.

As far as gtk4 goes ... although its already in development. The 4.0 version seems to be planned for spring 2019. But before gtk4 moves into its so called stable state, I think you can probably add at least one or two additional years to that.

(skipping gtk3 all together could also be a option. It would kinda trash the work you already did on it. But would free-up the time to complete it, in favor of venturing into gtk4)

you need to define two classes ...

I appreciate the vote of confidence in my presumed coding and compiling skills, but I'm sorry to say that compared to the real deal/coders, I'm not even coming close to what you would call an apprentice. (Complete C noob to boot)

... But I'm of course not the only one that might read this, I hope. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.