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

Organization chart: add new offsetDirection property to allow how offset property gets applied #15225

Closed
SteveSchreiner opened this issue Feb 25, 2021 · 15 comments · Fixed by #16285
Assignees
Labels
Type: Feature Request Used when a new feature is requested either directly or indirectly

Comments

@SteveSchreiner
Copy link

Description of the feature

Create a new organization chart property named "offsetDirection" that takes values of "vertical" or "horizontal". This would be used in conjunction with the existing "offset" property as current offset is only applied in fixed direction depending if chart is inverted or not. This would allow organization nodes to be shifted up or down, or left or right.

Library related to the feature

https://api.highcharts.com/highcharts/plotOptions.organization

Proof of Concept/Live example for the feature

https://jsfiddle.net/ttbsoftware/a5h1vwdp/

image

You can vote for this feature by adding a thumbs-up reaction to this post.

@SteveSchreiner SteveSchreiner added the Type: Feature Request Used when a new feature is requested either directly or indirectly label Feb 25, 2021
@highsoft-bot highsoft-bot added this to To do in Development-Flow Feb 25, 2021
@KacperMadej
Copy link

Hi @SteveSchreiner

Thank you for submitting the feature request.

Wouldn't it be more useful and simpler to add offset for the second dimension so you could use both horizontal and vertical offset at any time?

@SteveSchreiner
Copy link
Author

@KacperMadej Thanks for the feedback. Agreed that would even be more useful to add the second dimension.

@KacperMadej
Copy link

@SteveSchreiner

PoC demo for testing: https://jsfiddle.net/BlackLabel/x17jmzup/
Known limitations (a.k.a. todo list):

@KacperMadej KacperMadej moved this from To do to Inbox in Development-Flow Mar 4, 2021
@SteveSchreiner
Copy link
Author

Hi, we are ready to use this now, has it made it into a release? Also, the demo poc didnt indent the right side properly
image (10)

@SteveSchreiner
Copy link
Author

@KacperMadej Hi, I was curious to the progress of this feature request? Thanks

@KacperMadej KacperMadej moved this from Inbox to Backlog in Development-Flow Mar 29, 2021
@KacperMadej
Copy link

Bumping priority. I will post an update later this week.

@SteveSchreiner
Copy link
Author

@KacperMadej Checking in to see if you have an update? Thanks, Steve

@KacperMadej KacperMadej self-assigned this Apr 9, 2021
@KacperMadej
Copy link

The issue will be in progress first thing next week.

@KacperMadej KacperMadej moved this from Backlog to In progress in Development-Flow Apr 12, 2021
@SteveSchreiner
Copy link
Author

Thanks! My team is looking forward to this!

@SteveSchreiner
Copy link
Author

@KacperMadej Just checking in to see if there is an ETA for this? Thanks

@KacperMadej
Copy link

We are planning to finish implementation before the next release - in the next few weeks.
If you need this functionality sooner then we could improve the PoC demo - just let me know what should be improved, focused on, etc.

@SteveSchreiner
Copy link
Author

SteveSchreiner commented Apr 28, 2021

@KacperMadej Hmm, ok thought this would be ready sooner. You can see my issues with the POC in my screenshot from Mar 25th. Thanks

@KacperMadej
Copy link

horizontalOffset is already there - as shown for the move item in the original PoC demo.
A demo showing it for the last item: https://jsfiddle.net/BlackLabel/x17jmzup/2/

@SteveSchreiner
Copy link
Author

@KacperMadej Did this make it into today's release? https://www.highcharts.com/blog/changelog/

@KacperMadej
Copy link

No, it did not. That release was closed before the feature was fully passed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature Request Used when a new feature is requested either directly or indirectly
Projects
Development

Successfully merging a pull request may close this issue.

3 participants