thank you for your work there, some comments: - how likely is it that upstream will take that work? I guess that since they didn't approve grip as an optional depends and will not do so until it's done in a way the gtk team is happy with? If the patch will not go upstream soon please prefix the ubuntu specific functions to have an ubuntu_ prefix, it will make easier to figure in gdb trace if the issue is in ubuntu code for example and avoid some confusion - could you get the pinch to work without changing the default scroll wheel behaviour? it's quite handy for mouse use to be able to zoom and unzoom easily - if you want to change a default gsettings key don't patch the schemas this way, the way to do that is to add an eog.gsettings-override in the debian directory, see man dh_installgsettings
out of those the patch seems fine but breaking mouse usuability is sort of a blocker -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/823640 Title: Add uTouch gesture support to EOG To manage notifications about this bug go to: https://bugs.launchpad.net/ubuntu/+source/eog/+bug/823640/+subscriptions -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
