[nodebb-plugin-github-embed] GitHub Issue Embedding
-
In your example the #1370 is named first and #1328 after it, but the issues shown below are in reverse order... Otherwise nice done
Are you planning to integrate textcomplete into this?
-
And maybe you should add some styles that reduce the user-image size within "Recent Posts" widget, there is a giant julian-image laughing about me
-
@frissdiegurke said:
And maybe you should add some styles that reduce the user-image size within "Recent Posts" widget, there is a giant julian-image laughing about me
LOL you know, I noticed that too, but I wasn't sure whether it was a problem with core (since it auto adds
img-responsive
class to all images), or whether the problem was the plugin. I know we get around it in our emoji plugins with an!important
style, but I often wonder if there's a better way...
As for the issues being the wrong way around, that's a good point (leave it for my debut post a bug to show up, eh?). It's because I use two separate regular expressions to find matches. The shorthand regex is run first (hence why it showed up first). Perhaps they can be merged into one regular expression...
Text completion would be possible... didn't really think of it yet
-
@julian said:
LOL you know, I noticed that too, but I wasn't sure whether it was a problem with core (since it auto adds img-responsive class to all images), or whether the problem was the plugin. I know we get around it in our emoji plugins with an !important style, but I often wonder if there's a better way...
Isn't the reason that the divs
.github-issue
and.meta
are missing and because of this the selector.github-issue .meta img
doesn't match any more?
If you add.author-picture { float: left; width: 16px; margin-right: .5em; }
it should be better, shouldn't it (just tested temporarily addition within browser)?
And aclear: left;
should separate multiple embeds at least a bit -
@julian lol, got the last 3 notifications (1h ago) delivered with this one ^^ no clue why
-
@julian And I get 502 on accessing https://community.nodebb.org/notifications may there be some kind of bug?
-
@julian You're welcome it's always nice to look into other peoples code, if it's well-structured
And to test your plugin once more: #1413 -
gh#1337
Just testing this.
-
It's mostly a aesthetic choice. I'm not a fan of plugins that replace certain keywords with an inline object (take for example, youtube videos). That example is less of a problem, but I like being able to reference an issue in the middle of a sentence... a la:
Take a look at gh#256, that's where we solved that bug
^ No interruption of flow, and the referenced issue is at the bottom for you to check out if you want.
-
I'll disagree with both of you and suggest that the output be a popup similar to @Schamper's plugin. On mobile, just links to the actual issue. When I mention gh#123 it should be a link, at the very least. Just some ideas
EDIT: Scratch all of the above, you guys just gave me a great idea for a hook. Will get back to this later
-
@psychobunny reading the title of the issue you used in your example took 4 attempts to make any sense... kept thinking I was reading words over and over again like a child that reads the same line twice.