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

[WIP] Feature/Events #82

Closed
wants to merge 13 commits into from
Closed

[WIP] Feature/Events #82

wants to merge 13 commits into from

Conversation

friedger
Copy link
Contributor

@friedger friedger commented Dec 2, 2015

This PR adds handling of urls of the form

  • /event/:eventId
  • /:tag -> bad UI reusing the MainCtrl

screen shot 2015-12-02 at 11 27 12

@friedger
Copy link
Contributor Author

friedger commented Dec 2, 2015

related to #80 and #81

@Splaktar Splaktar self-assigned this Dec 4, 2015
…epeat`

Fix some styling issues (more to do still)
Refactor routing, components, and asset locations.
Remove empty files.
Added in some one time bindings for performance.
Filter in controller as much as possible for performance.
@Splaktar Splaktar changed the title Feature/Events [WIP] Feature/Events Dec 12, 2015
@Splaktar
Copy link
Contributor

Splaktar commented Dec 12, 2015

I put the big event lists in md-virtual-repeat containers and did some other performance and styling related things.
screen shot 2015-12-12 at 5 01 25 pm

Still working on this though. Need to investigate why the Hub is not returning color or description for tags when requested via JSONP instead of GET. Probably should look into writing up a $resource service for the Hub for use in this project.

Use minified version of ngGeolocation.
Bump version of ngGeolocation.
Sometimes this was being stored in the $scope which wasn't always visible and caused duplication of data.
Update to Angular Material 1.0.0.
@Splaktar
Copy link
Contributor

Issue of Hub not returning all data for tags has been resolved.

@Splaktar
Copy link
Contributor

This has been merged into #102 and will be closed when that is merged to master.

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

Successfully merging this pull request may close these issues.

2 participants