Skip to content

Added support for attributes on the Abbreviation element.#6

Merged
joshbruce merged 1 commit into8fold:masterfrom
Ambient-Impact:attributes
Apr 28, 2021
Merged

Added support for attributes on the Abbreviation element.#6
joshbruce merged 1 commit into8fold:masterfrom
Ambient-Impact:attributes

Conversation

@Ambient-Impact
Copy link
Contributor

This allows for both in-content attributes defined by content authors and also manipulation in event listeners.

@joshbruce
Copy link
Member

I like it. Would it be possible to add another test case to demonstrate use??

The expected and used string can be inside the test case itself.

@Ambient-Impact
Copy link
Contributor Author

Will do!

@Ambient-Impact
Copy link
Contributor Author

I don't have PHP 7.4 locally (only have 7.3), so bear with me through a couple force pushes to see if the test actually works. :P

@Ambient-Impact Ambient-Impact force-pushed the attributes branch 2 times, most recently from 3a59a4b to 96340a7 Compare April 27, 2021 20:23
@joshbruce
Copy link
Member

@Ambient-Impact: No worries on the PHP version. Also seems like the CI/CD wasn't set to run automatically - sorry about that.

Let me know when you think it's ready to go.

@Ambient-Impact
Copy link
Contributor Author

Just fixed a comment typo in the test. I don't see anything else missing so it should be fine on my end.

@joshbruce joshbruce merged commit 77e6c2e into 8fold:master Apr 28, 2021
@joshbruce
Copy link
Member

Thanks again @Ambient-Impact - taught me how to incorporate attributes into extensions as well!

@Ambient-Impact
Copy link
Contributor Author

You're welcome

@Ambient-Impact
Copy link
Contributor Author

Honestly, I'm still poking around and figuring out how CommonMark works for a project, so I'm learning as well. 😄

@joshbruce
Copy link
Member

joshbruce commented Apr 28, 2021

lol - Know how that goes. The CommonMark library is one of the ones we sponsor. The main developer seems pretty approachable.

I did add this to the CommonMark discussion board as well to see if they would consider it being an actual part of the spec as well.

If you haven't already, highly recommend either or both of them for resources.

@Ambient-Impact
Copy link
Contributor Author

I've talked a bit with the CommonMark people (Colin, Mark), and they do seem pretty helpful. Good luck with the spec stuff!

@Ambient-Impact Ambient-Impact deleted the attributes branch May 1, 2021 21:16
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.

2 participants