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.

Sign up for GitHub

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

Jump to bottom

polymer should support blur #12062

Closed
DartBot opened this issue Jul 26, 2013 · 10 comments
Closed

polymer should support blur #12062

DartBot opened this issue Jul 26, 2013 · 10 comments
Milestone
1.2

Comments

@DartBot
Copy link

DartBot commented Jul 26, 2013

This issue was originally filed by Erik.Grimes...@gmail.com

What steps will reproduce the problem?

  1. Create a polymer element with with multiple <input on-blur='blur' value='{{model.valueX}}'> elements and the
    method blur.
  2. Focus in and out of the input fields by tabbing or clicking between inputs.

What is the expected output? What do you see instead?

Expected: The blur method of the element should be called.
Actual: The blur method is not called and no error or warning is given.

What version of the product are you using? On what operating system?

Mac OS X 10.8.4
Dart Editor version 0.6.9_r25388
Dart SDK version 0.6.9.2_r25388

analyzer_experimental:
    description: analyzer_experimental
    source: hosted
    version: "0.6.5"
  args:
    description: args
    source: hosted
    version: "0.6.5"
  browser:
    description: browser
    source: hosted
    version: "0.6.5"
  csslib:
    description: csslib
    source: hosted
    version: "0.4.7+10"
  fancy_syntax:
    description: fancy_syntax
    source: hosted
    version: "0.0.4"
  html5lib:
    description: html5lib
    source: hosted
    version: "0.4.3"
  intl:
    description: intl
    source: hosted
    version: "0.6.5"
  js:
    description: js
    source: hosted
    version: "0.0.24"
  logging:
    description: logging
    source: hosted
    version: "0.6.5"
  mdv:
    description: mdv
    source: hosted
    version: "0.6.5"
  meta:
    description: meta
    source: hosted
    version: "0.6.5"
  observe:
    description: observe
    source: hosted
    version: "0.6.5"
  path:
    description: path
    source: hosted
    version: "0.6.5"
  pathos:
    description: pathos
    source: hosted
    version: "0.6.3+1"
  polymer:
    description: polymer
    source: hosted
    version: "0.5.0+1"
  serialization:
    description: serialization
    source: hosted
    version: "0.6.5"
  shadow_dom:
    description: shadow_dom
    source: hosted
    version: "0.6.3+2"
  source_maps:
    description: source_maps
    source: hosted
    version: "0.6.5"
  unittest:
    description: unittest
    source: hosted
    version: "0.6.5"

Please provide any additional information below.
@DartBot
Copy link
Author

DartBot commented Jul 26, 2013

This comment was originally written by Erik.G...@gmail.com

Here's the related web ui group discussion: https://groups.google.com/a/dartlang.org/forum/#!topic/web-ui/LzFkT__bkEA

@madsager
Copy link
Contributor

Added Area-WebUI, Triaged labels.

@sethladd
Copy link
Contributor

See also this sample for a repro: https://github.com/sethladd/dart-polymer-dart-examples/tree/master/web/simple_focus_blur_events

Removed Area-WebUI label.
Added Area-Polymer label.

@sethladd
Copy link
Contributor

Here's a sample JS file that shows the bug in polymer.js:

<html>
<head>
    <title>Test</title>
  <script src="polymer.min.js"></script>
</head>
<body>

<polymer-element name="my-test">
  <template>
    Type: <input type="text" on-blur="blurMe" on-focus="focusMe">
  </template>
  <script>
  Polymer('my-test', {
    focusMe: function(event, detail, target) {
      console.log('Focused!');
    },
    blurMe: function(event, detail, target) {
      console.log('Blurred');
    }
  });
  </script>
</polymer-element>

<my-test></my-test>
    
</body>
</html>

(using polymer.js from 10/10)

The bug over on the polymer side is: Polymer/polymer#208

@sigmundch
Copy link
Member

Added this to the Later milestone.

@sigmundch
Copy link
Member

Removed this from the Later milestone.
Added this to the M9 milestone.

@clayberg
Copy link

Removed this from the M9 milestone.
Added this to the 1.1 milestone.

@sigmundch
Copy link
Member

Removed this from the 1.1 milestone.
Added this to the 1.2 milestone.

@sigmundch
Copy link
Member

Hey Seth - I couldn't find that bug in the polymer repo. Was it moved somewhere else?

Added Library-Polymer label.

@jmesserly
Copy link

@Siggi -- Seth's link worked for me.

I think this might be fixed because they changed how the event binding works.
In any case, we have github issue to track, no need for a dupe here :)

Polymer/polymer#208

Added MovedToGithub label.

@DartBot DartBot added this to the 1.2 milestone May 13, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.2
Development

No branches or pull requests
7 participants
@sethladd @kevmoo @jmesserly @sigmundch @madsager @clayberg @DartBot