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

Update the descriptions of barThickness, offsetGridLines and offset #5600

Merged
merged 1 commit into from Jul 14, 2018

Conversation

nagix
Copy link
Contributor

@nagix nagix commented Jun 26, 2018

Doc improvements based on #4545 and #4994.

@nagix nagix changed the title Update the descriptions of barThickness, offsetGridLines ans offset Update the descriptions of barThickness, offsetGridLines and offset Jun 26, 2018
@nagix nagix force-pushed the issue-5600 branch 2 times, most recently from ba90ff1 to 4d36e5a Compare June 26, 2018 18:21
etimberg
etimberg previously approved these changes Jun 30, 2018
@@ -95,12 +95,19 @@ The bar chart defines the following configuration options. These options are mer
| ---- | ---- | ------- | -----------
| `barPercentage` | `Number` | `0.9` | Percent (0-1) of the available width each bar should be within the category width. 1.0 will take the whole category width and put the bars right next to each other. [more...](#barpercentage-vs-categorypercentage)
| `categoryPercentage` | `Number` | `0.8` | Percent (0-1) of the available width each category should be within the sample width. [more...](#barpercentage-vs-categorypercentage)
| `barThickness` | `Number` | | Manually set width of each bar in pixels. If not set, the base sample widths are calculated automatically so that they take the full available widths without overlap. Then, the bars are sized using `barPercentage` and `categoryPercentage`.
| `barThickness` | `Number/String` | | Manually set width of each bar in pixels. If set to `'flex'`, it computes "optimal" sample widths that globally arrange bars side by side. If not set (default), bars are equally sized based on the smallest interval. [more...](#barThickness)
Copy link
Member

Choose a reason for hiding this comment

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

I don't think that the #barThickness anchor works, it should be all lowercase

@simonbrunel simonbrunel added this to the Version 2.8 milestone Jul 1, 2018
@nagix
Copy link
Contributor Author

nagix commented Jul 3, 2018

The #barThickness anchor was fixed.

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

Successfully merging this pull request may close these issues.

None yet

4 participants