WordPress.org

Ready to get started?Download WordPress

Forums

Embed GitHub Gist
[Feature Request] Compose Gist, Hide Branding (2 posts)

  1. aoberoi
    Member
    Posted 2 years ago #

    First of all, I want to say THANKS for this plugin, its great :)

    I had two potential new features in mind. One would be to use the Gist API to actually help compose a Gist without leaving WordPress. I see that another plugin has accomplished this (http://wordpress.org/extend/plugins/simple-gist-embed/) and since the source is open this might be worth looking at for a head start. I think it could be done better in terms of UI. The second request would be a way to remove the Gist branding below the code as an optional attribute in the shortcode (via css or javascript at actual render time). This makes sense when you have a bunch of one line code snippets and you want to keep consistency with some larger bodies of code. I have come across this when writing tutorials on my blog.

    If I get a chance, I'd like to contribute code. Let me know if you have a public repository anywhere that I can push to if you are interested.

    http://wordpress.org/extend/plugins/embed-github-gist/

  2. dflydev
    Member
    Plugin Author

    Posted 2 years ago #

    I'm happy to accept patches. Please see:

    https://github.com/dflydev/embed-github-gist

    Here are my comments on your feature suggestions:

    Compose a Gist without leaving WordPress

    I'm less interested in this one. Mainly because the entire plugin more or less geared to referencing an existing Gist. It is simple and short and not complicated. People might have more reasons to ignore this plugin if it looks like it does too much and/or imposes a specific type of editing on the backend. I'm not saying I'll block it outright, but I think that if there is another plugin that does this (the one you referenced) maybe tweaking that plugin a bit to better meet your needs would be a better way to go?

    Remove the Gist branding below the code as an optional attribute in the shortcode

    I think that this would be fine. Make this functionality opt-in, though, instead of opt-out. :)

Topic Closed

This topic has been closed to new replies.

About this Plugin

About this Topic