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

embeds #246

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

embeds #246

wants to merge 6 commits into from

Conversation

janat08
Copy link

@janat08 janat08 commented Jun 17, 2019

fix #205
I added this into core plugins, and made a separate page in guides for docs.

It doesn't parse titles into sidebar since you'd have to sort out the order in which they should come, which the documentation will note as much. The controls for video/audio is a boolean attribute therefore it can't be overridden with additional attributes like controls='false', and it is set on by default.

@janat08
Copy link
Author

janat08 commented Nov 18, 2019

Please merge and reward

@janat08
Copy link
Author

janat08 commented Dec 6, 2019

@egoist?

@janat08
Copy link
Author

janat08 commented Dec 26, 2019

@egoist

@janat08
Copy link
Author

janat08 commented Feb 12, 2020

@egoist wouldn't mind just the bounty, pr review whenever you feel like it. Its been half a year.

@egoist
Copy link
Owner

egoist commented Mar 24, 2020

Actually we only need to use embed to inline external code, for iframe audio you can directly write HTML. See #205 (comment)

@janat08
Copy link
Author

janat08 commented May 30, 2020

Updated, please reward money at very long last.

@janat08
Copy link
Author

janat08 commented May 30, 2020

I can't at all tell whats up with the circleci, as it loads a loader with null>null>null for the workflow. I'll go fix another PR to see if youll deign to reward me with that one.

@janat08
Copy link
Author

janat08 commented May 30, 2020

So the circleCI is giving crap during linting that has nothing to do with my code, and doesn't even show up on local machine.

@janat08
Copy link
Author

janat08 commented Jun 9, 2020

?

1 similar comment
@janat08
Copy link
Author

janat08 commented Jun 23, 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.

embed code fragments from external sources
2 participants