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

feat(goal): expose max sizing limits in theme.goal options #1621

Merged
merged 5 commits into from
Mar 21, 2022

Conversation

nickofthyme
Copy link
Collaborator

@nickofthyme nickofthyme commented Mar 4, 2022

Summary

Exposes the sizing limitations for circular and bullet gauge and goal charts. Default values are same as original hard coded values to no functional changes.

interface Theme {
  ...
  goal: {
      ...
+    maxBulletSize: number;
+    maxCircularSize: number;
+    barThicknessMinSizeRatio: number;
+    baselineArcThickness: number;
+    baselineBarThickness: number;
+    marginRatio: number;
+    maxTickFontSize: number;
+    maxLabelFontSize: number;
+    maxCentralFontSize: number;
+    arcBoxSamplePitch: Radian;
+    capturePad: number;
  }
}

Issues

fix #1599

Checklist

  • The proper chart type label has been added (e.g. :xy, :partition)
  • The proper feature labels have been added (e.g. :interactions, :axis)
  • All related issues have been linked (i.e. closes #123, fixes #123)
  • New public API exports have been added to packages/charts/src/index.ts
  • Unit tests have been added or updated to match the most common scenarios

@nickofthyme nickofthyme added enhancement New feature or request :styling Styling related issue :goal/gauge (old) Old Goal/Gauge chart related issues labels Mar 4, 2022
@nickofthyme nickofthyme requested a review from markov00 March 4, 2022 22:53
@nickofthyme nickofthyme requested a review from monfera March 14, 2022 19:11
Copy link
Member

@markov00 markov00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@nickofthyme nickofthyme merged commit 60a14ba into elastic:master Mar 21, 2022
@nickofthyme nickofthyme deleted the max-goal-sizing branch March 21, 2022 16:27
nickofthyme pushed a commit that referenced this pull request Mar 29, 2022
# [45.1.0](v45.0.1...v45.1.0) (2022-03-29)

### Bug Fixes

* **axis:** ordinal number ending fix for the weekly resolution ([#1634](#1634)) ([18b4077](18b4077))
* **deps:** update dependency @elastic/eui to ^52.2.0 ([#1632](#1632)) ([7e0be07](7e0be07))
* **deps:** update dependency @elastic/eui to v50 ([#1622](#1622)) ([0eb7975](0eb7975))
* **deps:** update dependency @elastic/eui to v51 ([#1624](#1624)) ([64d87e5](64d87e5))
* **deps:** update dependency @elastic/eui to v52 ([#1630](#1630)) ([ada254e](ada254e))
* **goal:** chart placement and overlap issues ([#1620](#1620)) ([b5d375b](b5d375b))

### Features

* **goal:** expose max sizing limits in theme.goal options ([#1621](#1621)) ([60a14ba](60a14ba))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request :goal/gauge (old) Old Goal/Gauge chart related issues :styling Styling related issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Goal] Resizing limitations.
3 participants