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 (1,013 stars) (2,482 forks)batch import
good first issue

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

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
familiarity with Polymer data binding documentation
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
90
Research direction
Review the referenced PR #5125 in Polymer/polymer to understand the change. Then locate the data binding documentation page at polymer project.org/2.0/docs/devguide/data binding#attribute binding and add a note explaining that automatic capitalization detection works only for existing SVG attributes with capital letters, as detailed in the MDN SVG attribute reference.