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

Properties must be hyphenated in html #65

Merged
merged 1 commit into from May 18, 2018

Conversation

BennyAlex
Copy link
Contributor

No description provided.

@BennyAlex
Copy link
Contributor Author

@alessiomaffeis are you alive?

@lindbergtobias
Copy link

+1

@BennyAlex
Copy link
Contributor Author

@tobyryuk why thumbs down?

@tobyryuk
Copy link
Contributor

Because it was a super unnecessary comment since this PR is only for changing the README.

@BennyAlex
Copy link
Contributor Author

@tobyryuk I also published another pull request and @alessiomaffeis didn't respond

@tobyryuk
Copy link
Contributor

tobyryuk commented May 18, 2018

He has no obligation to respond to you though 😕 A tip would have been to write something like Any update on this? rather than are you alive?

@BennyAlex
Copy link
Contributor Author

No words...

@alessiomaffeis alessiomaffeis merged commit 073039f into alessiomaffeis:master May 18, 2018
@alessiomaffeis
Copy link
Owner

Yes, I am alive. But I also have a life.
All of this was pretty sad and unnecessary, both the thread and the PR, but I merged it anyway.

Have a nice weekend!

@BennyAlex
Copy link
Contributor Author

@alessiomaffeis
but the other PR isn't unnecessary.
But anyways, this whole component is crap, don't have nice code, many bugs and don't follow many vue patterns...

@tobyryuk
Copy link
Contributor

If there's anything you don't like in this component, maybe you should file a PR?

@BennyAlex
Copy link
Contributor Author

I already did so stfu

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.

None yet

4 participants