Skip to content
This repository has been archived by the owner on Apr 20, 2019. It is now read-only.

Add the doc for bind-property-event syntax #368

Closed
Andrzej-W opened this issue Jan 4, 2019 · 4 comments
Closed

Add the doc for bind-property-event syntax #368

Andrzej-W opened this issue Jan 4, 2019 · 4 comments
Assignees
Labels
🚀 Enhancement Enhance the topic 🚶‍♂️ P1 Normal priority
Projects

Comments

@Andrzej-W
Copy link
Contributor

Here dotnet/aspnetcore#5521 is a discussion about a bug in data binding. Notice the syntax: bind-value-oninput=@myField and a few other examples. It is not documented at all and should be added here: https://blazor.net/docs/components/index.html#data-binding

First, someone from Blazor team should tell us whether in the foreseeable future it will work as expected and what is really expected. Is this universal syntax in the form bind-property-event or only bind-value-event. It is probably designed only for HTML elements and not for user defined components. Which HTML elements?

  • input (what type? text, password, checkbox, etc., new HTML types: date, time, number, email, etc.)
  • textarea
  • select

cc: @SteveSandersonMS

@guardrex guardrex added 🚀 Enhancement Enhance the topic 🚶‍♂️ P1 Normal priority labels Jan 6, 2019
@guardrex guardrex added this to To do in Blazor.Docs via automation Jan 6, 2019
@guardrex guardrex added this to the 0.8.0 milestone Jan 6, 2019
@guardrex
Copy link
Collaborator

guardrex commented Jan 6, 2019

Thanks @Andrzej-W ... I'll wait until we hear from @SteveSandersonMS on the questions you asked. However, some quick testing should also be able to determine what's supported if I get to this before he responds. I might have some time next weekend to take a look at this issue.

@guardrex guardrex self-assigned this Jan 6, 2019
@Andrzej-W
Copy link
Contributor Author

@guardrex Quick testing in Blazor 0.7 is unfortunately not enough - usually you will get Error: System.InvalidCastException: Specified cast is not valid. (dotnet/aspnetcore#5521 (comment))

@guardrex
Copy link
Collaborator

guardrex commented Jan 7, 2019

Yes, I mean that it "isn't supported" if an error is thrown, so we wouldn't document anything for the current release that doesn't work.

@guardrex
Copy link
Collaborator

guardrex commented Feb 2, 2019

Issue as been moved to aspnet/Docs.

@guardrex guardrex closed this as completed Feb 2, 2019
Blazor.Docs automation moved this from To do to Done Feb 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🚀 Enhancement Enhance the topic 🚶‍♂️ P1 Normal priority
Projects
Blazor.Docs
  
Done
Development

No branches or pull requests

2 participants