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

Side by Side Diff: chrome/browser/resources/shared/js/cr/ui/table.js

Issue 8608007: cr/ui/list.js: Support rows with variable heights. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Omit the custom lead item height Created 9 years 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 (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 /** 5 /**
6 * @fileoverview This implements a table control. 6 * @fileoverview This implements a table control.
7 */ 7 */
8 8
9 cr.define('cr.ui', function() { 9 cr.define('cr.ui', function() {
10 const ListSelectionModel = cr.ui.ListSelectionModel; 10 const ListSelectionModel = cr.ui.ListSelectionModel;
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
102 * 102 *
103 * @type {boolean} 103 * @type {boolean}
104 */ 104 */
105 get autoExpands() { 105 get autoExpands() {
106 return this.list_.autoExpands; 106 return this.list_.autoExpands;
107 }, 107 },
108 set autoExpands(autoExpands) { 108 set autoExpands(autoExpands) {
109 this.list_.autoExpands = autoExpands; 109 this.list_.autoExpands = autoExpands;
110 }, 110 },
111 111
112 get fixedHeight() {
113 return this.list_.fixedHeight;
114 },
115 set fixedHeight(fixedHeight) {
116 this.list_.fixedHeight = fixedHeight;
117 },
118
112 /** 119 /**
113 * Returns render function for row. 120 * Returns render function for row.
114 * @return {Function(*, cr.ui.Table): HTMLElement} Render function. 121 * @return {Function(*, cr.ui.Table): HTMLElement} Render function.
115 */ 122 */
116 getRenderFunction: function() { 123 getRenderFunction: function() {
117 return this.list_.renderFunction_; 124 return this.list_.renderFunction_;
118 }, 125 },
119 126
120 /** 127 /**
121 * Sets render function for row. 128 * Sets render function for row.
(...skipping 161 matching lines...) Expand 10 before | Expand all | Expand 10 after
283 * because table contents can contain controls that can be focused, and for 290 * because table contents can contain controls that can be focused, and for
284 * some purposes (e.g., styling), the table can still be conceptually focused 291 * some purposes (e.g., styling), the table can still be conceptually focused
285 * at that point even though it doesn't actually have the page focus. 292 * at that point even though it doesn't actually have the page focus.
286 */ 293 */
287 cr.defineProperty(Table, 'hasElementFocus', cr.PropertyKind.BOOL_ATTR); 294 cr.defineProperty(Table, 'hasElementFocus', cr.PropertyKind.BOOL_ATTR);
288 295
289 return { 296 return {
290 Table: Table 297 Table: Table
291 }; 298 };
292 }); 299 });
OLDNEW
« no previous file with comments | « chrome/browser/resources/shared/js/cr/ui/list_selection_model.js ('k') | chrome/browser/resources/task_manager/main.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698