From ed95c8c88f12cce49ba5dcf046fd39be76a1b0eb Mon Sep 17 00:00:00 2001 From: yumoqing Date: Fri, 29 May 2026 13:58:42 +0800 Subject: [PATCH] debug: use console.log instead of bricks.debug for tabular click tracing --- bricks/tabular.js | 9 ++++++--- dist/bricks.js | 9 ++++++--- 2 files changed, 12 insertions(+), 6 deletions(-) diff --git a/bricks/tabular.js b/bricks/tabular.js index 67acfa0..9dbfdc7 100644 --- a/bricks/tabular.js +++ b/bricks/tabular.js @@ -20,9 +20,11 @@ bricks.Tabular = class extends bricks.DataViewer { } async build_record_view(record){ var r = await this.build_info(record); + console.log('[Tabular] build_record_view', 'row widget=', r, 'dom=', r.dom_element); if (!this.content_view){ r.user_data = record; r.bind('click', this.record_clicked.bind(this, r, record)); + console.log('[Tabular] build_record_view: click bound to DataRow'); return r; } var row = new bricks.VBox({ @@ -34,6 +36,7 @@ bricks.Tabular = class extends bricks.DataViewer { }); row.add_widget(content); r.bind('click', this.record_clicked.bind(this, row, record)); + console.log('[Tabular] build_record_view: click bound to VBox wrapper'); row.rec_widget = r; row.content_widget = content; row.user_data = record; @@ -41,11 +44,11 @@ bricks.Tabular = class extends bricks.DataViewer { } async record_clicked(row, record, event){ var r = event.target.bricks_widget; - bricks.debug('record_clicked', 'row=', row, 'record=', record, 'target=', event.target, 'targetWidget=', r); + console.log('[Tabular] record_clicked', 'row=', row, 'record=', record, 'target=', event.target.tagName, 'targetWidget=', r); // Prevent click on interactive elements (links, buttons, inputs) from triggering selection var target = event.target; if (target && (target.tagName === 'A' || target.tagName === 'BUTTON' || target.tagName === 'INPUT' || target.tagName === 'SELECT' || target.tagName === 'TEXTAREA')) { - bricks.debug('record_clicked: skip interactive element', target.tagName); + console.log('[Tabular] record_clicked: skip interactive element', target.tagName); return; } @@ -67,7 +70,7 @@ bricks.Tabular = class extends bricks.DataViewer { } else { row.set_css('tabular-row-selected'); } - bricks.debug('record_clicked: row selected, dispatching row_selected'); + console.log('[Tabular] record_clicked: row selected, dispatching row_selected'); this.dispatch('row_selected', row.user_data); } } diff --git a/dist/bricks.js b/dist/bricks.js index 4e31e54..1b463e4 100644 --- a/dist/bricks.js +++ b/dist/bricks.js @@ -13012,9 +13012,11 @@ bricks.Tabular = class extends bricks.DataViewer { } async build_record_view(record){ var r = await this.build_info(record); + console.log('[Tabular] build_record_view', 'row widget=', r, 'dom=', r.dom_element); if (!this.content_view){ r.user_data = record; r.bind('click', this.record_clicked.bind(this, r, record)); + console.log('[Tabular] build_record_view: click bound to DataRow'); return r; } var row = new bricks.VBox({ @@ -13026,6 +13028,7 @@ bricks.Tabular = class extends bricks.DataViewer { }); row.add_widget(content); r.bind('click', this.record_clicked.bind(this, row, record)); + console.log('[Tabular] build_record_view: click bound to VBox wrapper'); row.rec_widget = r; row.content_widget = content; row.user_data = record; @@ -13033,11 +13036,11 @@ bricks.Tabular = class extends bricks.DataViewer { } async record_clicked(row, record, event){ var r = event.target.bricks_widget; - bricks.debug('record_clicked', 'row=', row, 'record=', record, 'target=', event.target, 'targetWidget=', r); + console.log('[Tabular] record_clicked', 'row=', row, 'record=', record, 'target=', event.target.tagName, 'targetWidget=', r); // Prevent click on interactive elements (links, buttons, inputs) from triggering selection var target = event.target; if (target && (target.tagName === 'A' || target.tagName === 'BUTTON' || target.tagName === 'INPUT' || target.tagName === 'SELECT' || target.tagName === 'TEXTAREA')) { - bricks.debug('record_clicked: skip interactive element', target.tagName); + console.log('[Tabular] record_clicked: skip interactive element', target.tagName); return; } @@ -13059,7 +13062,7 @@ bricks.Tabular = class extends bricks.DataViewer { } else { row.set_css('tabular-row-selected'); } - bricks.debug('record_clicked: row selected, dispatching row_selected'); + console.log('[Tabular] record_clicked: row selected, dispatching row_selected'); this.dispatch('row_selected', row.user_data); } }