Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(95)

Side by Side Diff: ui/file_manager/file_manager/foreground/elements/files_quick_view.js

Issue 2259433004: Quick View: fixed the bug that 'No preview available' text is not shown. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: nit Created 4 years, 4 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
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 var FilesQuickView = Polymer({ 5 var FilesQuickView = Polymer({
6 is: 'files-quick-view', 6 is: 'files-quick-view',
7 7
8 properties: { 8 properties: {
9 // File media type, e.g. image, video. 9 // File media type, e.g. image, video.
10 type: String, 10 type: String,
11 filePath: String, 11 filePath: String,
12 // URLs should be accessible from webview since contets are rendered inside 12 // URLs should be accessible from webview since contets are rendered inside
13 // it. Hint: use URL.createObjectURL. 13 // it. Hint: use URL.createObjectURL.
14 contentUrl: String, 14 contentUrl: String,
15 videoPoster: String, 15 videoPoster: String,
16 audioArtwork: String, 16 audioArtwork: String,
17 autoplay: Boolean, 17 autoplay: Boolean,
18 18
19 // metadata-box-active-changed event is fired on attribute change. 19 // metadata-box-active-changed event is fired on attribute change.
20 metadataBoxActive: { 20 metadataBoxActive: {
21 value: true, 21 value: true,
22 type: Boolean, 22 type: Boolean,
23 notify: true, 23 notify: true,
24 } 24 },
25 // Text shown when no playback is available.
26 noPlaybackText: String,
27 // Text shown when no preview is available.
28 noPreviewText: String,
25 }, 29 },
26 30
27 listeners: { 31 listeners: {
28 'iron-overlay-closed': 'clear', 32 'iron-overlay-closed': 'clear',
29 'files-safe-media-tap-outside': 'close', 33 'files-safe-media-tap-outside': 'close',
30 }, 34 },
31 35
32 // Clears fields. 36 // Clears fields.
33 clear: function() { 37 clear: function() {
34 this.type = ''; 38 this.type = '';
(...skipping 115 matching lines...) Expand 10 before | Expand all | Expand 10 after
150 * @param {string} type 154 * @param {string} type
151 * @return {boolean} 155 * @return {boolean}
152 * 156 *
153 * @private 157 * @private
154 */ 158 */
155 isUnsupported_: function(type) { 159 isUnsupported_: function(type) {
156 return !this.isImage_(type) && !this.isVideo_(type) && !this.isAudio_(type); 160 return !this.isImage_(type) && !this.isVideo_(type) && !this.isAudio_(type);
157 }, 161 },
158 162
159 }); 163 });
OLDNEW
« no previous file with comments | « ui/file_manager/file_manager/foreground/elements/files_quick_view.html ('k') | ui/file_manager/file_manager/main.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698