-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
Comprehensive Events Page #378
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
Merged
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
9c1ec66
Code for pulling in event data and intial, terrible, display code.
mikeal f3b4328
change events lists to three column layout
rnsloan 0839d95
Fixing data, adding the rest of meetup.com
mikeal 0f9e896
Adding a map to the top of the events page. Also formatting fixes to …
mikeal a0c7ebf
Moving to basic streets.
mikeal f68aaaa
Improved map views with legend.
mikeal e3f17e3
Defaulting map to near where you are.
mikeal 912a629
move events map css to .styl . Adapt for mobile. Move JS to bottom of…
rnsloan dd94b03
add has-js class to html element for css targeting
rnsloan efbad69
add toggle display of events
rnsloan e203d04
replace hard-coded event numbers
rnsloan 54690a0
Better tooltips in map and real counts.
mikeal 83aff10
Fixing merge conflict.
mikeal 13e19e3
consistent JS semi-colons. css cleanup.
rnsloan 12a23cf
More data and information on being listed.
mikeal f796fbf
Adding image for Node.js Interactive.
mikeal File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Fixing data, adding the rest of meetup.com
- Loading branch information
commit 0839d95b1f07d49b914d60c84201f3c2b82a4ab6
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CO: 'Latin America'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I only have all of the ones in here that we need for the meetups currently coming out of meetup.com, I wonder why there isn't one showing up for Colombia.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like they need to be tagged with the topic?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right now they need to be tagged with "nodejs" and they need to have "node" in the name, but we may remove that second part later.