Skip to content
This repository has been archived by the owner on Jul 29, 2019. It is now read-only.

[Timeline] margin configuration option is "broken" #2626

Closed
mvdam opened this issue Jan 24, 2017 · 11 comments
Closed

[Timeline] margin configuration option is "broken" #2626

mvdam opened this issue Jan 24, 2017 · 11 comments

Comments

@mvdam
Copy link

mvdam commented Jan 24, 2017

When upgrading from v4.16.1 to v4.17.0 / v4.18.0 the margin configuration option seems to be "broken". Timeline items that were nicely aligned after each other are now pushed to a new row. See screenshots below;

At v4.16.1 it works fine!
v4 16 1

When upgrading to v4.17.0 / v4.18.0 the layout is somewhat broken.
v4 18 0

We use the following configuration for "margin".

margin: { axis: 5, item : { vertical: 0, horizontal: 0 } }

Although the margin options still work, you cannot use a value of 0 to prevent items from being pushed to a new row.

@yotamberk
Copy link
Contributor

yotamberk commented Jan 24, 2017 via email

@yotamberk
Copy link
Contributor

yotamberk commented Jan 24, 2017 via email

@mvdam
Copy link
Author

mvdam commented Jan 24, 2017

Thank you, your solution works. Is it intended behaviour to use a negative value? Seems more like a hack to me?

@yotamberk
Copy link
Contributor

yotamberk commented Jan 24, 2017 via email

@mvdam mvdam closed this as completed Jan 24, 2017
@mvdam mvdam reopened this Jan 24, 2017
@mojoaxel
Copy link
Member

It is kind of a hack. There is a bug that i couldnt solve elegantly so in the meantime this solves it

@yotamberk Can you please refer to the issue number of that bug.

@mojoaxel
Copy link
Member

@mvdam By the way: You timeline looks realy nicely styled. Is there any chance that you could provide a public example with this style? A would realy like to add something like this to our showcases or our examples.

@yotamberk
Copy link
Contributor

duplicate of #2318
@mojoaxel I remember referring to the PR that introduced this bug. I'll look for it again. I didn't have an elegant way of solving it so I left it as is at the time...

@yotamberk
Copy link
Contributor

found it: #2107

@mojoaxel
Copy link
Member

@mvdam See #2318 (comment)
closing as duplicate of #2318

@mvdam
Copy link
Author

mvdam commented Jan 31, 2017

@mojoaxel Unfortunately I can't give you a live example of our timeline. However if you wish I can create lookalike for the examples page?

@mojoaxel
Copy link
Member

Unfortunately I can't give you a live example of our timeline. However if you wish I can create lookalike for the examples page?

@mvdam That would be amazing! At the moment almost all Timelines look the same. If you can com up with a nice styled example I'm more than happy to add it to our examples or showcases.

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

No branches or pull requests

3 participants