Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

Datepicker calendar #608

Merged
merged 23 commits into from
Apr 26, 2017
Merged

Datepicker calendar #608

merged 23 commits into from
Apr 26, 2017

Conversation

Blackbaud-PatrickOFriel
Copy link
Contributor

@Blackbaud-PatrickOFriel Blackbaud-PatrickOFriel commented Apr 20, 2017

Calendar component for #141. Temporary docs aren't in the main components page, but can be found at components/datepicker

Things to think about:

  - Instead of using Javascript Date, think about using skyDate class
with day, month, year properties.
  - Get everything working with simplest case
TODO:
  - Add styles for month picker and year picker
  - clean up styles
  - verify startingDay
  - verify minDate, maxDate
  - clean up code
  - unit tests
  - visual tests
  - remove demo (only show demo for input)
TODO:
  - create dateObject interface
  - unit tests
TODO:
  - set active function tests
  - edge case tests
  - rudimentary documentation
  - remove docs from main component page until connected with input
  - visual tests
  - accessibility
TODO:
  - fix tests for year picker
  - add a bunch of tests for keyboard functionality
  - initial docs
  - remove docs from component list
  - visual tests
@codecov-io
Copy link

codecov-io commented Apr 20, 2017

Codecov Report

Merging #608 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #608    +/-   ##
======================================
  Coverage     100%   100%            
======================================
  Files         269    277     +8     
  Lines        4298   4629   +331     
  Branches      459    511    +52     
======================================
+ Hits         4298   4629   +331
Impacted Files Coverage Δ
src/modules/list/list-data-response.model.ts 100% <ø> (ø) ⬆️
src/modules/list/list-data.provider.ts 100% <ø> (ø) ⬆️
...odules/datepicker/datepicker-calendar.component.ts 100% <100%> (ø)
src/modules/datepicker/monthpicker.component.ts 100% <100%> (ø)
src/modules/datepicker/datepicker.module.ts 100% <100%> (ø)
src/modules/datepicker/yearpicker.component.ts 100% <100%> (ø)
...rc/modules/datepicker/datepicker-config.service.ts 100% <100%> (ø)
src/modules/datepicker/daypicker.component.ts 100% <100%> (ø)
src/modules/datepicker/date-formatter.ts 100% <100%> (ø)
.../datepicker/datepicker-calendar-inner.component.ts 100% <100%> (ø)
... and 8 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e369b84...904b316. Read the comment docs.

Copy link
Contributor

@Blackbaud-ToddRoberts Blackbaud-ToddRoberts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hover state of dates/month/arrows should have borders to look like hover state of secondary buttons

@Blackbaud-PatrickOFriel
Copy link
Contributor Author

I'll get Todd's style changes in this afternoon and merge it in

@Blackbaud-PatrickOFriel Blackbaud-PatrickOFriel merged commit 9a58c26 into master Apr 26, 2017
@Blackbaud-PatrickOFriel Blackbaud-PatrickOFriel deleted the datepicker-calendar branch June 16, 2017 15:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants