Skip to content
This repository has been archived by the owner on Dec 19, 2017. It is now read-only.

bind-required doesn't work on <paper-input>s #331

Closed
DartBot opened this issue Jun 5, 2015 · 3 comments
Closed

bind-required doesn't work on <paper-input>s #331

DartBot opened this issue Jun 5, 2015 · 3 comments
Labels

Comments

@DartBot
Copy link

DartBot commented Jun 5, 2015

Originally opened as dart-lang/sdk#21460

This issue was originally filed by [email protected]


Use case:
I would like to dynamically set whether a <paper-input> is required by using "bind-required." This is using a polymer js <paper-input> with a angulardart backing model.

e.g.
<paper-input type="text" bind-required="myRequiredBool"></paper-input>

Expected result:
The paper input is set to required if the variable is true, and not otherwise.

Actual result:
The paper input is always has required set to false.

If I change the attribute to "bind-disabled," the disabled attribute toggles as expected, so it looks like bind-required isn't being hooked up correctly.

Please include the following information:

  • Dart SDK version: 1.6.0-dev.9.0
  • polymer package version: v0.2
  • Operating system: Goobuntu Trusty
  • Browser (if any): Chrome Version 38.0.2125.111 (64-bit)
@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/2049220?v=3" align="left" width="48" height="48"hspace="10"> Comment by sigmundch


So far, we have not been able to reproduce the issue.

We are not sure if you are using polymer.js alone or also the Dart wrappers with polymer.dart. Jake was able to bind the required field when he used the latest versions of the Dart packages (polymer.dart 0.15.1+3 and the Dart wrappers in paper_elements 0.5.0).

When you say version v0.2 of polymer, do you mean polymer.js? That version sounds a bit outdated too (they are currently on version 0.4.2 I believe). So it's possible that updating to the latest version will fix the issue for you, if not, the problem could be in the angular hooks to interact with polymer elements (If so, you can let them know in their issue tracker: https://github.com/angular/angular.dart/ so they can investigate).


Added NeedsInfo label.

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

This comment was originally written by [email protected]


We are not sure if you are using polymer.js alone or also the Dart wrappers with polymer.dart.
polymer.js only

When you say version v0.2 of polymer, do you mean polymer.js?
Yes. However, I think I was actually mistaken when I put v0.2 as the version number, and I think I'm actually already on v0.4.2.

I guess this might be a problem on the angular.dart side, since we had a different problem with <paper-radio-button>s and "bind-selected" that ended up being angular issue: dart-archive/angular.dart#1449

I will file a new bug for angular.dart (sorry, wasn't sure where to file this, and someone suggested filing to here). :)

@DartBot
Copy link
Author

DartBot commented Jun 5, 2015

<img src="https://avatars.githubusercontent.com/u/2049220?v=3" align="left" width="48" height="48"hspace="10"> Comment by sigmundch


No worries - glad to hear we are narrowing down where the issue could be :)

I just saw the issue you created over there, so I'll close this bug for now and mark it as a duplicate of dart-archive/angular.dart#1588, but feel free to reply here if there is something else we can help with on our end.


Added MovedToGithub label.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

No branches or pull requests

1 participant