From b4bd3b66e71a41b165105253fed0511bdf1a669b Mon Sep 17 00:00:00 2001 From: M03M Date: Tue, 17 Mar 2015 14:23:45 -0500 Subject: [PATCH 1/3] Made EnhancedButton's onKeyboardFocus prop optional. --- src/js/date-picker/calendar.jsx | 1 - src/js/date-picker/date-picker-dialog.jsx | 2 +- src/js/date-picker/date-picker.jsx | 1 + src/js/date-picker/day-button.jsx | 6 +++++- src/js/enhanced-button.jsx | 11 +++++------ src/js/flat-button.jsx | 2 +- src/js/floating-action-button.jsx | 2 +- src/js/icon-button.jsx | 2 +- src/js/raised-button.jsx | 2 +- 9 files changed, 16 insertions(+), 13 deletions(-) diff --git a/src/js/date-picker/calendar.jsx b/src/js/date-picker/calendar.jsx index 195cf11599f2f3..491e047b3fff22 100644 --- a/src/js/date-picker/calendar.jsx +++ b/src/js/date-picker/calendar.jsx @@ -160,7 +160,6 @@ var Calendar = React.createClass({ _setSelectedDate: function(d) { var newDisplayDate = DateTime.getFirstDayOfMonth(d); - if (newDisplayDate !== this.state.displayDate) { this._setDisplayDate(newDisplayDate, d); } else { diff --git a/src/js/date-picker/date-picker-dialog.jsx b/src/js/date-picker/date-picker-dialog.jsx index 723c40fae1f7d4..bf9e1d3ef96253 100644 --- a/src/js/date-picker/date-picker-dialog.jsx +++ b/src/js/date-picker/date-picker-dialog.jsx @@ -54,7 +54,7 @@ var DatePickerDialog = React.createClass({ fontSize: '14px', color: CustomVariables.datePickerCalendarTextColor }, - + dialogContents: { width: this.props.mode === 'landscape' ? '560px' : '280px' } diff --git a/src/js/date-picker/date-picker.jsx b/src/js/date-picker/date-picker.jsx index bf294e6efd6e6a..ff25f8ff456f22 100644 --- a/src/js/date-picker/date-picker.jsx +++ b/src/js/date-picker/date-picker.jsx @@ -68,6 +68,7 @@ var DatePicker = React.createClass({ onTouchTap={this._handleInputTouchTap} /> Date: Tue, 17 Mar 2015 14:32:42 -0500 Subject: [PATCH 2/3] Refactored css for date picker day button --- src/js/date-picker/day-button.jsx | 54 +++++++++++++++---- src/less/components/components.less | 1 - .../components/date-picker/date-picker.less | 2 - .../components/date-picker/day-button.less | 35 ------------ src/less/variables/custom-variables.less | 4 +- 5 files changed, 45 insertions(+), 51 deletions(-) delete mode 100644 src/less/components/date-picker/date-picker.less delete mode 100644 src/less/components/date-picker/day-button.less diff --git a/src/js/date-picker/day-button.jsx b/src/js/date-picker/day-button.jsx index 5852da618bf258..72235a7859a8dc 100644 --- a/src/js/date-picker/day-button.jsx +++ b/src/js/date-picker/day-button.jsx @@ -1,11 +1,13 @@ var React = require('react'); -var Classable = require('../mixins/classable'); +var StylePropable = require('../mixins/style-propable'); +var Transition = require('../styles/mixins/transitions'); +var CustomVariables = require('../styles/variables/custom-variables'); var DateTime = require('../utils/date-time'); var EnhancedButton = require('../enhanced-button'); var DayButton = React.createClass({ - mixins: [Classable], + mixins: [StylePropable], propTypes: { date: React.PropTypes.object, @@ -21,22 +23,52 @@ var DayButton = React.createClass({ selected, ...other } = this.props; - var classes = this.getClasses('mui-date-picker-day-button', { - 'mui-is-current-date': DateTime.isEqualDate(this.props.date, new Date()), - 'mui-is-selected': this.props.selected - }); + + var styles = { + root: { + position: 'relative', + float: 'left', + width: 36, + padding: '4px 2px', + }, + + label: { + position: 'relative', + }, + + select: { + position: 'absolute', + height: 32, + width: 32, + opacity: 0, + borderRadius: '50%', + transform: 'scale(0)', + transition: Transition.easeOut(), + backgroundColor: CustomVariables.datePickerSelectColor, + }, + }; + + if (this.props.selected) { + styles.label.color = CustomVariables.datePickerSelectTextColor; + styles.select.opacity = 1; + styles.select.transform = 'scale(1)'; + if (DateTime.isEqualDate(this.props.date, new Date())) { + styles.root.color = CustomVariables.datePickerColor; + } + } return this.props.date ? ( -
- {this.props.date.getDate()} + onTouchTap={this._handleTouchTap} + onKeyboardFocus={this._handleKeyboardFocus}> +
+ {this.props.date.getDate()} ) : ( - + ); }, diff --git a/src/less/components/components.less b/src/less/components/components.less index 04cad2f5119629..e3aae64c969f1a 100644 --- a/src/less/components/components.less +++ b/src/less/components/components.less @@ -1,3 +1,2 @@ -@import "date-picker/date-picker.less"; @import "paper.less"; @import "toolbar.less"; \ No newline at end of file diff --git a/src/less/components/date-picker/date-picker.less b/src/less/components/date-picker/date-picker.less deleted file mode 100644 index 2586ca17089146..00000000000000 --- a/src/less/components/date-picker/date-picker.less +++ /dev/null @@ -1,2 +0,0 @@ - -@import "day-button.less"; diff --git a/src/less/components/date-picker/day-button.less b/src/less/components/date-picker/day-button.less deleted file mode 100644 index 61e7fc3292be9d..00000000000000 --- a/src/less/components/date-picker/day-button.less +++ /dev/null @@ -1,35 +0,0 @@ -.mui-date-picker-day-button { - position: relative; - float: left; - width: 36px; - padding: 4px 2px; - - .mui-date-picker-day-button-select { - position: absolute; - background-color: @date-picker-select-color; - height: 32px; - width: 32px; - opacity: 0; - border-radius: 50%; - transform: scale(0); - .ease-out(); - } - - .mui-date-picker-day-button-label { - position: relative; - } - - &.mui-is-selected { - .mui-date-picker-day-button-label { - color: @date-picker-select-text-color; - } - .mui-date-picker-day-button-select { - opacity: 1; - transform: scale(1); - } - } - - &.mui-is-current-date { - color: @date-picker-color; - } -} \ No newline at end of file diff --git a/src/less/variables/custom-variables.less b/src/less/variables/custom-variables.less index e0ade7eb1b0264..38059fc68e5e81 100644 --- a/src/less/variables/custom-variables.less +++ b/src/less/variables/custom-variables.less @@ -30,8 +30,8 @@ //Disabled Colors @disabled-color: fade(@body-text-color, 30%); -//Date Picker -@date-picker-color: @primary-1-color; +// //Date Picker + @date-picker-color: @primary-1-color; @date-picker-text-color: @white; @date-picker-calendar-text-color: @body-text-color; @date-picker-select-color: @primary-2-color; From 5d2b7d162385c7da25ceb030f41de9ed139c2627 Mon Sep 17 00:00:00 2001 From: M03M Date: Tue, 17 Mar 2015 14:35:13 -0500 Subject: [PATCH 3/3] Forgot to remove unused datepicker variables. --- src/less/variables/custom-variables.less | 7 ------- 1 file changed, 7 deletions(-) diff --git a/src/less/variables/custom-variables.less b/src/less/variables/custom-variables.less index 38059fc68e5e81..dfbfa79b0f4bca 100644 --- a/src/less/variables/custom-variables.less +++ b/src/less/variables/custom-variables.less @@ -30,13 +30,6 @@ //Disabled Colors @disabled-color: fade(@body-text-color, 30%); -// //Date Picker - @date-picker-color: @primary-1-color; -@date-picker-text-color: @white; -@date-picker-calendar-text-color: @body-text-color; -@date-picker-select-color: @primary-2-color; -@date-picker-select-text-color: @white; - // buttons @button-height: 36px; @button-min-width: 88px;