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

GTK3 port and ZIM 0.71 #15

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

gutierri
Copy link
Contributor

fixs: #14 #13 #12

There are some issues with this implementation, such as:

  • The Ctrl + Shift + E key shortcut does not work and goes into "Read-Only" mode or something like that (But via menu: Tools -> Log Entry, it works). We can change to Ctrl + Shift + J.

  • This implementation only works on newer versions. Is keeping compatibility with older versions of ZIM an idea?

@gutierri gutierri changed the title WIP: GTK3 port and ZIM 7.1 WIP: GTK3 port and ZIM 0.71 Jul 21, 2019
@domnpp
Copy link

domnpp commented Aug 18, 2019

@gutierri You definetly did better job than me. Great! Using your version now. It works, needs journal plugin enabled and configured correclty.

domnpp added a commit to domnpp/zim-desktop-wiki that referenced this pull request Aug 18, 2019
Osndok/zim-plugin-nowbutton#15

Run configurations, gitgnore for my special setup (make unit tests work, files must be known to git).
domnpp added a commit to domnpp/zim-desktop-wiki that referenced this pull request Aug 18, 2019
Osndok/zim-plugin-nowbutton#15

Run configurations, gitgnore for my special setup (make unit tests work, files must be known to git).
@gutierri
Copy link
Contributor Author

@domenn Great! I am also working on a patch to maintain backward compatibility. This plugin has helped me too.

@gutierri gutierri changed the title WIP: GTK3 port and ZIM 0.71 GTK3 port and ZIM 0.71 May 13, 2020
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

Successfully merging this pull request may close these issues.

3 participants