Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[mixed time-series] Secondary y-axis title is disguised/misplaced #18235

Open
3 tasks done
rumbin opened this issue Jan 31, 2022 · 1 comment
Open
3 tasks done

[mixed time-series] Secondary y-axis title is disguised/misplaced #18235

rumbin opened this issue Jan 31, 2022 · 1 comment

Comments

@rumbin
Copy link
Contributor

@rumbin rumbin commented Jan 31, 2022

The Mixed Time-Series chart type allows for configuring the title of the primary and the secondary y-axis.
However, while only the title of the primary axis is shown next to the axis, the title of the secondary one is placed at the upper end of the axis where it gets hidden by bar values and zoom controls.

How to reproduce the bug

  1. Create a mixed time-series chart
  2. Configure axis titles for both y-axes
  3. Check the Data Zoom checkbox

Expected results

Both axes carry the titles as configured

Actual results

Only the left/primary y-axis has a title shown at the center of the axis.
The title of the right/secondary axis is misplaced at the end of the axis and thus overprinted/disquised by bar values and/or zoom controls.

Screenshots

mixed_time-series_y2_title

Environment

  • browser type and version: Chrome 96.0.4664.93
  • superset version: 1.4.0
  • all other versions: as defined in Superset's Docker image
  • Feature flags:
# stable:
      "THUMBNAILS": True,
      "SQLLAB_BACKEND_PERSISTENCE": True,
      "ENABLE_TEMPLATE_PROCESSING": True,
      "DASHBOARD_CROSS_FILTERS": True,

      # experimental:
      "ALERT_REPORTS": True,
      "ALERTS_ATTACH_REPORTS": True,
      "DASHBOARD_NATIVE_FILTERS": True,

      # unclassified:
      "ENABLE_EXPLORE_DRAG_AND_DROP": True,
      "ENABLE_DND_WITH_CLICK_UX": True,

      # development:
      "DASHBOARD_NATIVE_FILTERS_SET": True

Checklist

Make sure to follow these steps before submitting your issue - thank you!

  • I have checked the superset logs for python stacktraces and included it here as text if there are any.
  • I have reproduced the issue with at least the latest released version of superset.
  • I have checked the issue tracker for the same issue and I haven't found one similar.
@villebro
Copy link
Member

@villebro villebro commented Feb 8, 2022

Repro'd - I started working on a fix but I identified multiple inconsistencies in this area, so will need to circle back slightly later due to lack of time right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants