Skip to content

fix(explore): remove extra spacing when Advanced Analytics section is hidden#37456

Merged
msyavuz merged 1 commit intoapache:masterfrom
VanessaGiannoni:fix/incorrect-chart-spacing-on-control-panel
Jan 29, 2026
Merged

fix(explore): remove extra spacing when Advanced Analytics section is hidden#37456
msyavuz merged 1 commit intoapache:masterfrom
VanessaGiannoni:fix/incorrect-chart-spacing-on-control-panel

Conversation

@VanessaGiannoni
Copy link
Contributor

@VanessaGiannoni VanessaGiannoni commented Jan 26, 2026

SUMMARY

Fixed incorrect spacing in the control panel when the Advanced Analytics section is hidden for non-temporal x-axis charts.

Root cause: The displayTimeRelatedControls visibility function correctly determined when to hide Advanced Analytics for non-temporal x-axis, but the section rendering used visibility: hidden instead of display: none, leaving empty space in the UI.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

AFTER

Screenshot 2026-01-26 at 19 05 15

BEFORE

Screenshot 2026-01-26 at 19 12 04

TESTING INSTRUCTIONS

  1. Create or edit a chart with a non-temporal x-axis (e.g., categorical data)
  2. Open the chart in Explore view
  3. Verify the Advanced Analytics section is not visible in the control panel
  4. Confirm there is no extra spacing gap where the section would normally appear
  5. Switch to a chart with a temporal x-axis and verify Advanced Analytics section appears normally
  6. Run the existing tests: npm run test -- --testPathPatterns="ControlPanelsContainer.test.tsx"

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

@bito-code-review
Copy link
Contributor

bito-code-review bot commented Jan 26, 2026

Code Review Agent Run #07374b

Actionable Suggestions - 0
Review Details
  • Files reviewed - 1 · Commit Range: 302efec..302efec
    • superset-frontend/src/explore/components/ControlPanelsContainer.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

AI Code Review powered by Bito Logo

@dosubot dosubot bot added change:frontend Requires changing the frontend explore Namespace | Anything related to Explore labels Jan 26, 2026
@netlify
Copy link

netlify bot commented Jan 26, 2026

Deploy Preview for superset-docs-preview ready!

Name Link
🔨 Latest commit 302efec
🔍 Latest deploy log https://app.netlify.com/projects/superset-docs-preview/deploys/6977e4a405c77e0008af9193
😎 Deploy Preview https://deploy-preview-37456--superset-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@rusackas rusackas requested review from EnxDev and msyavuz January 27, 2026 18:23
@msyavuz
Copy link
Member

msyavuz commented Jan 28, 2026

Is before after screenshots backwards?

@VanessaGiannoni
Copy link
Contributor Author

Is before after screenshots backwards?

Ooops my bad, let me fix the labeling 😅

@msyavuz msyavuz merged commit 7110fc9 into apache:master Jan 29, 2026
85 of 87 checks passed
sadpandajoe pushed a commit that referenced this pull request Jan 29, 2026
@sadpandajoe sadpandajoe added the v6.0 Label added by the release manager to track PRs to be included in the 6.0 branch label Jan 29, 2026
sadpandajoe pushed a commit that referenced this pull request Jan 29, 2026
LevisNgigi pushed a commit to LevisNgigi/superset that referenced this pull request Feb 4, 2026
aminghadersohi pushed a commit to aminghadersohi/superset that referenced this pull request Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

change:frontend Requires changing the frontend explore Namespace | Anything related to Explore size/XS v6.0 Label added by the release manager to track PRs to be included in the 6.0 branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants