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 works with Gedit 3.16.3 #10

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

Conversation

R1kk3r
Copy link

@R1kk3r R1kk3r commented Sep 29, 2015

Make it works with Gedit of the last Debian testing update. For me, it is Gedit 3.16.3 but it can be compatible with other versions as well (not tested).

Quite similar to pull requests #9 but support internationalization and have a stateful menu entry.

Remove unused code
Add some basic comments
Optimize a bit (avoid lookup for action, don't destroy window but hide instead)
Remove unused code
Add some basic comments
Optimize a bit (avoid lookup for action, don't destroy window but hide instead)
@marc1uk
Copy link

marc1uk commented Jan 26, 2017

On gedit 3.14:
During handling of the above exception, another exception occurred:

Traceback (most recent call last):
File "/home/marc/.local/share/gedit/plugins/markdown-preview/init.py", line 25, in
from gi.repository import Gio, Gdk, Gtk, Gedit, GObject, WebKit, GLib
File "/usr/lib/python3/dist-packages/gi/importer.py", line 53, in find_module
'introspection typelib not found' % namespace)
ImportError: cannot import name WebKit, introspection typelib not found

(gedit:6669): libpeas-WARNING **: Error loading plugin 'markdown-preview'

@sixtyfive
Copy link

So this PR also suffers from the problem of python-webkit being unmaintained. Perhaps close it, then?

@darkdragon-001
Copy link

I just want to let you know, that I created a fork: https://github.com/darkdragon-001/gedit-markdown/

Please test the latest release. Pull requests are always welcome!

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.

4 participants