chrome/browser/resources/settings/people_page/change_picture.html - Issue 1575543003: Settings People Revamp: Implement parts of ChromeOS Change Picture

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: chrome/browser/resources/settings/people_page/change_picture.html

Issue 1575543003: Settings People Revamp: Implement parts of ChromeOS Change Picture (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: fix camera presence observer Created 4 years, 11 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

« chrome/browser/resources/settings/people_page/change_picture.css ('K') | « chrome/browser/resources/settings/people_page/change_picture.css ('k') | chrome/browser/resources/settings/people_page/change_picture.js » ('j') | chrome/browser/resources/settings/people_page/change_picture.js » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
(Empty)
	1 <link rel="import" href="chrome://resources/html/i18n_behavior.html">

	2 <link rel="import" href="chrome://resources/polymer/v1_0/polymer/polymer.html">

	3 <link rel="import" href="chrome://resources/polymer/v1_0/paper-button/paper-butt on.html">

	4 <link rel="import" href="chrome://md-settings/people_page/change_picture_private _api.html">

	5

	6 <dom-module id="settings-change-picture">

	7 <link rel="import" type="css"

	8 href="chrome://md-settings/settings_shared.css">

	9 <link rel="import" type="css" href="change_picture.css">

	10 <template>

	11 <div class="settings-box">

	12 <div class="split">

	13 <div id="availableIcons">

	14 <paper-button toggles

	15 active="{{isActiveImage_(profileImageUrl_, selectedImageUrl_)}}"

	16 on-tap="onProfileImageTap_">

	17 <img class="user-image" src="[[profileImageUrl_]]"

	18 alt="[[i18n('profilePhotoLoading')]]">

	19 </paper-button>

	20 <template is="dom-if" if="[[oldImageUrl_]]">

	21 <paper-button toggles

	22 active="{{isActiveImage_(oldImageUrl_, selectedImageUrl_)}}"

	23 on-tap="onOldImageTap_">

	24 <img class="user-image" src="[[oldImageUrl_]]">

	25 </paper-button>

	26 </template>

	27 <template is="dom-repeat" items="[[defaultImages_]]">

	28 <paper-button toggles

	29 active="{{isActiveImage_(item.url, selectedImageUrl_)}}"

	30 on-tap="onDefaultImageTap_" data-image-url$="[[item.url]]">
	dpapad 2016/01/11 19:25:50 data-image-url$="[[item.url]] Have not seen this data-image-url$="[[item.url]] Have not seen this syntax before. Is the "$" a typo, or intentional? michaelpg 2016/01/11 20:38:05 https://www.polymer-project.org/1.0/docs/devguide/ Show quoted text On 2016/01/11 19:25:50, dpapad wrote: > data-image-url$="[[item.url]] > > Have not seen this syntax before. Is the "$" a typo, or intentional? https://www.polymer-project.org/1.0/docs/devguide/data-binding.html#attribute...
	31 <img class="user-image" src="[[item.url]]" alt="[[item.title]]">

	32 </paper-button>

	33 </template>

	34 </div>

	35 </div>

	36 </div>

	37 </template>

	38 <script src="change_picture.js"></script>

	39 </dom-module>

OLD	NEW