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
added appReducer; created toggle feature, refactored component code #22
Conversation
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 tabs spacing is inconsistent
frontend/src/App.js
Outdated
|
||
class App extends Component { | ||
render() { | ||
console.log(this.props.view) |
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.
Remove console logs
frontend/src/reducers/appReducer.js
Outdated
export default (state = initialState, action) => { | ||
switch (action.type) { | ||
case CHANGE_VIEW: | ||
console.log(state.view); |
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.
remove console log
frontend/src/reducers/index.js
Outdated
@@ -1,6 +1,8 @@ | |||
import { combineReducers } from 'redux'; | |||
import emptyReducer from './empty'; |
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.
Now that we actually have a normal reducer, could you go ahead and delete the empty reducer?
frontend/src/reducers/appReducer.js
Outdated
import { CHANGE_VIEW } from '../actions/actionsTypes'; | ||
import initialState from './initialState'; | ||
|
||
export default (state = initialState, action) => { |
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.
Naming of this file is a little confusing. Could you name all reducers individual to their functions?
frontend/src/actions/appActions.js
Outdated
@@ -0,0 +1,5 @@ | |||
import * as types from './actionsTypes'; | |||
|
|||
export function changeView(appView){ |
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.
Could you name the file with reference to the function of the actions? i.e changeViewAction.js
frontend/src/constants/constants.js
Outdated
@@ -0,0 +1,2 @@ | |||
export const MAP_VIEW = "MAP_VIEW"; |
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.
Could you name the file more relevant to the content? having constants.js file inside a constants folder is a little confusing
* added appReducer; created toggle feature, refactored component code (#22) * added appReducer; created toggle feature, refactored component code * added color to denote selected toggle * added unselected color * removed loggin; renamed files for readability * added appReducer; created toggle feature, refactored component code (#22) * added appReducer; created toggle feature, refactored component code * added color to denote selected toggle * added unselected color * removed loggin; renamed files for readability * added initial setup for a searchbar * added all basic components to Sidebar * added full seach bar layout - refactored sidebar * added final structure of basic side bar * added basic ListViewCard to ListView * added mutiple ListViewCards * added footer position; added more styling * resized mapview to account for sidebar * added code for ListViewCards and styling * added icons to listViewCard * added appReducer; created toggle feature, refactored component code (#22) * added appReducer; created toggle feature, refactored component code * added color to denote selected toggle * added unselected color * removed loggin; renamed files for readability * added initial setup for a searchbar * added all basic components to Sidebar * added full seach bar layout - refactored sidebar * added final structure of basic side bar * added basic ListViewCard to ListView * added mutiple ListViewCards * added footer position; added more styling * resized mapview to account for sidebar * added code for ListViewCards and styling * added icons to listViewCard * changed login dialog to icon button; moved icon to search bar * fized uncuaght merge conflict * moved all assets into public * renamed login logo to keep styling * merge again
…or service provider. (#47) * added appReducer; created toggle feature, refactored component code (#22) * added appReducer; created toggle feature, refactored component code * added color to denote selected toggle * added unselected color * removed loggin; renamed files for readability * added bases for store selectCensusDivision reducer, highlight census division when clicked * binded onClick function; lost ability to track census division highlighting * added onFeature function to each census division * commit untracked action and reducer files * added highlight features for clicking * Added service provider population on map. * Removing unused components and C files? * Moved marker population function to improve performance. * Added popup for service provider markers. * Renaming of selectCensusDivision to changeSelectedCensusDivision.
No description provided.