Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(timeline): add markers to custom time bars #86

Merged
merged 3 commits into from
Sep 12, 2019
Merged

feat(timeline): add markers to custom time bars #86

merged 3 commits into from
Sep 12, 2019

Conversation

ryamaguchi0220
Copy link
Contributor

@ryamaguchi0220 ryamaguchi0220 commented Sep 12, 2019

@@ -1225,6 +1225,16 @@ <h2 id="Methods">Methods</h2>
</td>
</tr>

<tr>
Copy link
Member

Choose a reason for hiding this comment

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

CustomTime is a timeline feature. This docs should be added here instead: https://github.com/visjs/vis-timeline/blob/master/docs/timeline/index.html#L1554

Copy link
Contributor Author

@ryamaguchi0220 ryamaguchi0220 Sep 12, 2019

Choose a reason for hiding this comment

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

Thank you for the review.
Fixed in 9eac76f.

Copy link
Contributor Author

@ryamaguchi0220 ryamaguchi0220 Sep 12, 2019

Choose a reason for hiding this comment

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

I've fixed commit messages and this PR's title as well.

Copy link
Member

Choose a reason for hiding this comment

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

❤️ Thx!

@ryamaguchi0220 ryamaguchi0220 changed the title feat(graph2d): add markers to graphs feat(timeline): add markers to graphs Sep 12, 2019
@ryamaguchi0220 ryamaguchi0220 changed the title feat(timeline): add markers to graphs feat(timeline): add markers to custom time bars Sep 12, 2019
@mojoaxel mojoaxel merged commit 808c5a0 into visjs:master Sep 12, 2019
@mojoaxel
Copy link
Member

@ryamaguchi0220 Thanks again for you first contribution 🥇

@ryamaguchi0220 ryamaguchi0220 deleted the issue/84/marker branch September 13, 2019 01:52
Glandos added a commit to Glandos/vue2vis that referenced this pull request Jul 27, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Graph2d] Add markers which indicate what happened at the time to graphs
2 participants