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

Overlapping Y-axis labels #5156

Closed
boriskogan81 opened this issue Jan 17, 2018 · 8 comments
Closed

Overlapping Y-axis labels #5156

boriskogan81 opened this issue Jan 17, 2018 · 8 comments

Comments

@boriskogan81
Copy link

I have this Y-axis object:

{
	id: 'events',
	type: 'category',
	labels: this.state.eventLabelArray),
	scaleLabel: {
		display: true,
		labelString: 'Phenology'
	},
	ticks: {
		autoSkip: true
	}
}

Despite autoSkip being declared as "true," it's not working-each label is rendered, none are skipped:

image

If I de-duplicate the labels, the data is no longer mapped in the tooltip:

{
	id: 'events',
	type: 'category',
	labels: Array.from([...new Set(this.state.eventLabelArray)]),
	scaleLabel: {
		display: true,
		labelString: 'Phenology'
	},
	ticks: {
		autoSkip: true
	}
}

image

@etimberg
Copy link
Member

I believe the issue here is that the label auto skipper is not implemented for vertical axes. I think another ticket already covers this but I can't find it

@ximex
Copy link

ximex commented Feb 18, 2019

i also got this issue if i set a custom min/max value in the ticks object and the min/max value is only a little bit above/under the auto labeled values

@benmccann
Copy link
Contributor

I'm pretty sure the auto skipper is implemented for vertical axes:

var isHorizontal = me.isHorizontal();
. But it sounds like maybe there's a bug where it's not working correctly

@kurkle
Copy link
Member

kurkle commented Feb 18, 2019

auto skipper is enabled for vertical in master only (PR #5922), so probably fixed.

@ximex
Copy link

ximex commented Feb 18, 2019

is there a date for the next release?

@benmccann
Copy link
Contributor

TBD. I think we'd like to get a number of pending PRs merged first

@kurkle
Copy link
Member

kurkle commented Nov 15, 2019

Was this resolved?

@etimberg
Copy link
Member

Going to call this resolved since 2.9 shipped with these changes

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

No branches or pull requests

5 participants