-
Notifications
You must be signed in to change notification settings - Fork 743
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
Treat boolean attributes as HTML properties #188
Conversation
Unit test please :) |
@skyrpex why did you stop? Make unit tests please. I think your changes are useful for readonly or disabled attributes. I hope false value does not output attribute. |
This should do it. |
Guys sorry about not relative question, but who is able to review PRs on this repo? |
Sigh... |
This is good for custom attributes. If you care about W3C standards, check this PR instead. |
Copy/pasting from my comment in #252 @andrzejkupczyk When it mentions 'the empty string' that also allows it to be omitted. You can see examples of boolean attributes here W3C Boolean Attributes
The information about omitting empty string values is mentioned in the secion A Quick Introduction to HTML about 4 paragraphs in (emphasis mine)
|
👍 On Tue, Oct 25, 2016, 12:07 AM Andrew Smith notifications@github.com
|
It looks like this change is not available on 5.3. Is this intentional? |
HTML properties shouldn't have value. Attributes with boolean values will now be represented as HTML properties.
Given this Blade snippet:
It will compile to: