Polymer/old-docs-site

Add note for automatic capitalization detection of attribute bindings

Open

#2,490 opened on Mar 7, 2018

View on GitHub
 (1 comment) (0 reactions) (0 assignees)HTML (2,482 forks)batch import
good first issue

Repository metrics

Stars
 (1,013 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

In https://github.com/Polymer/polymer/pull/5125 we are fixing bindings to HTML attributes that have capitalization. The resulting change is the following:

Rather than the broken viewBox$="[[binding]]", Polymer will now automatically set an attribute when encountering viewBox="[[binding]]". This will only work if the attribute maintains capitalization. This is only the case for existing attributes, which mostly live in the SVG namespace: https://developer.mozilla.org/en-US/docs/Web/SVG/Attribute

This clarifying note should probably be added to https://www.polymer-project.org/2.0/docs/devguide/data-binding#attribute-binding

Contributor guide

Research direction
Locate the page at https://www.polymer project.org/2.0/docs/devguide/data binding#attribute binding and add a note explaining that only existing SVG attributes (as listed on MDN) preserve capitalization when using `attributeName="[[binding]]"` syntax.
Tech stack
htmljavascript
Domain
documentation
Issue type
Documentation
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
1
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Under 1 hour
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
understanding of Polymer and HTML data binding
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
90