fix(config): Wire LOGO_TARGET_PATH and document custom spinner usage

[shashbha14]

SUMMARY
This PR addresses user confusion regarding customizing the loading icon and resolves a bug where LOGO_TARGET_PATH was being ignored by the default theme system in Superset 6.0.0.

Specifically, it:

1. Documents Custom Loading Icon: Explicitly documents brandSpinnerUrl in THEME_DEFAULT within config.py. This clarifies that users can set a custom GIF/image loader via config without needing to replace source files or rebuild assets.
2. Fixes Logo Link: Updates THEME_DEFAULT to correctly use LOGO_TARGET_PATH for brandLogoHref. Previously, LOGO_TARGET_PATH was ignored by the default theme, forcing users to override the entire theme object just to change the logo link.
3. Clarifies Configuration: Added comments to APP_ICON, LOGO_TOOLTIP, LOGO_RIGHT_TEXT, and APP_ICON_WIDTH (deprecation note) to clarify their interaction with THEME_DEFAULT or their deprecated status.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
N/A (Configuration and Documentation changes only)

TESTING INSTRUCTIONS
To verify these changes locally:

1. Verify Logo Link Fix:

   • In superset_config.py, set LOGO_TARGET_PATH = "https://custom.url".
   • Restart Superset and click the top-left logo.
   • Result: It should navigate to https://custom.url (previously it would stay on / unless the theme was overridden).

2. Verify Custom Spinner:

   • In superset_config.py, set:

     THEME_DEFAULT = {
         "token": {
             "brandSpinnerUrl": "/static/assets/images/my-custom-spinner.gif"
         }
     }

   • Restart Superset.
   • Result: The custom GIF should appear during loading.

ADDITIONAL INFORMATION

• Has associated issue: Fixes Theming Error: Facing issue in Changing App Icon and Loading icon in Apache Superset 6.0.0 #36940
• 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]

Code Review Agent Run #3272be

Actionable Suggestions - 0

Review Details

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

  • superset/config.py
• Files skipped - 0
• Tools

  • Whispers (Secret Scanner) - ✔︎ Successful
  • Detect-secrets (Secret Scanner) - ✔︎ Successful
  • MyPy (Static Code Analysis) - ✔︎ Successful
  • Astral Ruff (Static Code Analysis) - ✔︎ 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 [email protected].

Documentation & Help

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

AI Code Review powered by

[netlify]

✅ Deploy Preview for superset-docs-preview ready!

Name	Link
🔨 Latest commit	4e6b533
🔍 Latest deploy log	https://app.netlify.com/projects/superset-docs-preview/deploys/695e4b3c25d1df00089b71ec
😎 Deploy Preview	https://deploy-preview-36951--superset-docs-preview.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

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