fix(explore): remove leftover debug console.log in ZoomConfigControl

[jbbqqf]

SUMMARY

Drops a leftover debug console.log('now in onbasewidthcahnge') (note the typo) from onBaseWidthChange in ZoomConfigControl.tsx. The statement fires on every drag of the Base Width slider in the Explore view, polluting the browser console during normal usage. Pure cleanup — no behavior change.

Fixes #39622 — Leftover Debug Statement Polluting Console in ZoomConfigControl

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Not applicable — the only observable change is the absence of one line in the DevTools console while dragging the Base Width slider.

TESTING INSTRUCTIONS

Reproduce BEFORE/AFTER yourself (copy-paste)

A reviewer can confirm by inspection — only the exact message changes between BEFORE and AFTER.

# --- one-time setup ---
git clone https://github.com/apache/superset.git /tmp/repro && cd /tmp/repro

# --- BEFORE (origin/master) ---
git checkout origin/master
grep -n "onbasewidthcahnge" superset-frontend/src/explore/components/controls/ZoomConfigControl/ZoomConfigControl.tsx
# Expected: line 69 prints `console.log('now in onbasewidthcahnge');`

# --- AFTER (this PR) ---
git fetch https://github.com/jbbqqf/superset.git feat/39622-remove-zoomconfig-debug-log
git checkout FETCH_HEAD
grep -n "onbasewidthcahnge" superset-frontend/src/explore/components/controls/ZoomConfigControl/ZoomConfigControl.tsx
# Expected: no output; the debug statement is gone.

# Optional UI verification (requires the superset-frontend dev server):
#   cd superset-frontend && npm ci && npm run dev-server
# Open Explore on a chart that uses ZoomConfigControl, drag the Base Width
# slider, and inspect the browser DevTools console.
# BEFORE: 'now in onbasewidthcahnge' fires on every drag event.
# AFTER : the console stays clean during dragging.

What I ran locally

• grep -n "console\." superset-frontend/src/explore/components/controls/ZoomConfigControl/ — confirmed the file no longer has any console.* calls after the change.
• The repository's frontend ESLint config sets no-console: 'warn' (not error), which is why this leftover slipped through CI; this PR does not change that policy.
• No regression test added: this is a pure-cleanup one-line removal with no behavioral divergence to assert. The grep above is the equivalence check.

ADDITIONAL INFORMATION

#	Scenario	Input	Expected	Verified by
1	Default behavior	drag Base Width slider	console stays clean	grep on file confirms line removed

• Has associated issue: Leftover Debug Statement Polluting Console in ZoomConfigControl #39622
• Required feature flags:
• Changes UI (only in the sense of console output during dev usage; visible chart behavior is unchanged)
• Includes DB Migration
• Introduces new feature or API
• Removes existing feature or API

Risk / blast radius

Zero. The removed line was a console.log with no surrounding control flow. setBaseWidth(width) and the rest of onBaseWidthChange are untouched.

Removed a leftover debug console.log from the Base Width slider in ZoomConfigControl.

---

PR drafted with assistance from Claude Code. The change was reviewed manually against apache/superset's source. The reproducer block above was used during development; it is the same one a reviewer can paste verbatim.

[bito-code-review]

Code Review Agent Run #d727f8

Actionable Suggestions - 0

Review Details

• Files reviewed - 1 · Commit Range: 4de19fb..4de19fb

  • superset-frontend/src/explore/components/controls/ZoomConfigControl/ZoomConfigControl.tsx
• Files skipped - 0
• Tools

  • Whispers (Secret Scanner) - ✔︎ Successful
  • Detect-secrets (Secret Scanner) - ✔︎ Successful

---

Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

• /review - Manually triggers a full AI review.

• /pause - Pauses automatic reviews on this pull request.

• /resume - Resumes automatic reviews.

• /resolve - Marks all Bito-posted review comments as resolved.

• /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Superset You can customize the agent settings here or contact your Bito workspace admin at evan@preset.io.

Documentation & Help

• Customize agent settings
• Review rules
• General documentation
• FAQ

AI Code Review powered by

[rusackas]

Thanks for the fix! Will merge if/when CI turns green.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.83%. Comparing base (f67dd4a) to head (4de19fb).
⚠️ Report is 27 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master   #39991   +/-   ##
=======================================
  Coverage   63.83%   63.83%           
=======================================
  Files        2589     2589           
  Lines      137821   137820    -1     
  Branches    31928    31928           
=======================================
+ Hits        87978    87979    +1     
+ Misses      48327    48325    -2     
  Partials     1516     1516           

Flag	Coverage Δ
javascript	66.53% <ø> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.