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

Dfki branch update with all changes from mcq-dev #634

Open
wants to merge 53 commits into
base: dfki
Choose a base branch
from

Conversation

sosias
Copy link

@sosias sosias commented Mar 17, 2020

No description provided.

sosias and others added 30 commits October 16, 2019 16:52
…onConfig file or if doesn't exist from the default config
+ brand colors
+ sass reintroduction
+ brand colors
+ nav bar heights
+ animated timeframe radio select (day, week, etc.)
+ added placeholder images for d3 charts
... 
+ design adjustments to how the diary appears
+ first adjustments to the chart
+ some adjustments to tab bottom nav
+ reformatted trip detail template. Also added / changed some info like header date etc.
+ row-col format for dashboard with new lineChart nvd3 chart
+ new icons for tab navigation
updating colors, navbar
changing axis
creating the donut diagram
generally understanding D3
+ added var for default text color
+ tabs and nvd3 css changes
+ changed order or css import because they didn't make any sense
+ added storeModeToTrip function to store a mode given a trip object and the mode to be stored (As opposed to the previous function that relied on the trip being set in a global var when opening the mode selection popup)
+ more changes on the chart
+ added scope vars for querying last x days of metrics
+ removed title bars in our three views because they're unnecessary in combination with the bold titles inside the bottom tabs
+ made mode selection actually work in the diary list view and horizontally scrollable
+ got the timeframe selection with ion-tabs to actually work in dashboard
+
...according to co2 factor table received from DFKI
@AmplabJenkins
Copy link

Can one of the admins verify this patch?

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.

5 participants