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

Timeline component #4

Merged
merged 10 commits into from
Feb 6, 2019
Merged

Timeline component #4

merged 10 commits into from
Feb 6, 2019

Conversation

j8seangel
Copy link
Collaborator

@j8seangel j8seangel commented Feb 5, 2019

Moving time bar component to this repo #1

image

  • Created a sandbox page to test the time bar.
  • Added a simple router to navigate between components.

Nice to think about:

@j8seangel j8seangel requested a review from nerik February 5, 2019 11:38
@j8seangel j8seangel added the WIP label Feb 5, 2019
@j8seangel j8seangel removed the WIP label Feb 5, 2019
Copy link
Contributor

@nerik nerik left a comment

Choose a reason for hiding this comment

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

Good stuff 👍
We need to add clearer documentation to the README:

  • how to initialize and use the sandbox
  • brief description of what does each component

@@ -3,10 +3,10 @@
"version": "0.1.0",
"private": true,
"dependencies": {
"@globalfishingwatch/map-components": "link:..",
Copy link
Contributor

Choose a reason for hiding this comment

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

🤯

@@ -1 +1,5 @@
export { default as MiniGlobe } from './miniglobe/miniglobe'
export { default as Timebar } from './timebar/Timebar'
export { default as TimebarActivity } from './timebar/charts/Activity'
Copy link
Contributor

Choose a reason for hiding this comment

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

Not 100% sure about that. I'd rather being able to do

import Timebar from '@globalfishingwatch/map-components'
import { VesselEvents } from '@globalfishingwatch/map-components/timebarCharts'

than

import {
  Timebar,
  VesselEvents
} from '@globalfishingwatch/map-components'

@nerik
Copy link
Contributor

nerik commented Feb 5, 2019

Any idea how to fix this?
screenshot 2019-02-05 at 17 43 09

@j8seangel
Copy link
Collaborator Author

Any idea how to fix this?
screenshot 2019-02-05 at 17 43 09

don't understand neither why it happens sometimes... so I decided to create to create an .env file
and add the SKIP_PREFLIGHT_CHECK=true to skip that check

…p-components into timeline-component

* 'timeline-component' of github.com:GlobalFishingWatch/map-components:
  clarify docs
@j8seangel
Copy link
Collaborator Author

j8seangel commented Feb 6, 2019

Added components description and uglify dist code to make it lighter in production.

@j8seangel j8seangel merged commit 551fca7 into master Feb 6, 2019
@j8seangel j8seangel deleted the timeline-component branch February 6, 2019 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants