Default Y Axis Title Margin of 15 often overlaps with axis value text #32528

Activity

1. sfirke

   added

   requires:design-reviewRequires input/approval from a DesignerRequires input/approval from a Designer

   on Mar 6, 2025
2. dosubot

   added

   #bug:cosmeticCosmetic/layout/design tweak neededCosmetic/layout/design tweak needed

   on Mar 6, 2025
3. sfirke commented on Mar 6, 2025

   

   sfirke

   on Mar 6, 2025

   MemberAuthor

   More actions

   I wonder if we just change this value?

   superset/superset-frontend/packages/superset-ui-chart-controls/src/sections/chartTitle.tsx

   Line 87 in 82595df

   default: TITLE_MARGIN_OPTIONS[0],

   to default: TITLE_MARGIN_OPTIONS[1], instead of [0] to make it 30?
4. rusackas commented on Mar 6, 2025

   

   rusackas

   on Mar 6, 2025

   Member

   More actions

   Sounds reasonable to me! Simple PR to boot! No migrations required, and I wouldn't constitute it a breaking change.
5. rusackas

   added

   review:checkpointLast PR reviewed during the daily review standupLast PR reviewed during the daily review standup

   on Mar 6, 2025
6. rea725 commented on Mar 6, 2025

   

   rea725

   on Mar 6, 2025

   More actions

   
7. sfirke

   added

   good first issueGood first issues for new contributorsGood first issues for new contributors

   on Mar 6, 2025
8. mistercrunch commented on Mar 7, 2025

   

   mistercrunch

   on Mar 7, 2025

   Member

   More actions

   I wish echarts was more responsive here, or maybe we're not using it right (?) Anyhow bumping the default seems like a good place to start.
9. rusackas

   removed

   review:checkpointLast PR reviewed during the daily review standupLast PR reviewed during the daily review standup

   on Mar 10, 2025
10. Vitor-Avila

    closed this as completedin #32720on Mar 20, 2025