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

feat(datepicker): add ability to show one month for daterangepicker depend on max/min date value #5667

Merged
merged 6 commits into from
Mar 2, 2020

Conversation

daniloff200
Copy link
Contributor

close #3595

PR Checklist

Before creating new PR, please take a look at checklist below to make sure that you've done everything that needs to be done before we can merge it.

  • read and followed the CONTRIBUTING.md guide.
  • built and tested the changes locally.
  • added/updated tests.
  • added/updated API documentation.
  • added/updated demos.

@daniloff200 daniloff200 force-pushed the show-one-month-in-daterangepicker branch from 756ec0f to cc13370 Compare February 18, 2020 18:59
IraErshova
IraErshova previously approved these changes Feb 21, 2020
IraErshova
IraErshova previously approved these changes Feb 21, 2020
@codecov
Copy link

codecov bot commented Feb 21, 2020

Codecov Report

Merging #5667 into development will increase coverage by 0.12%.
The diff coverage is 82.69%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development    #5667      +/-   ##
===============================================
+ Coverage         73.5%   73.63%   +0.12%     
===============================================
  Files              322      323       +1     
  Lines            11595    11647      +52     
  Branches          2472     2481       +9     
===============================================
+ Hits              8523     8576      +53     
+ Misses            1983     1966      -17     
- Partials          1089     1105      +16
Impacted Files Coverage Δ
demo/src/ng-api-doc.ts 100% <ø> (ø) ⬆️
src/datepicker/bs-datepicker-inline.component.ts 67.94% <100%> (+0.41%) ⬆️
.../datepicker/bs-daterangepicker-inline.component.ts 62.35% <100%> (+22.09%) ⬆️
src/popover/popover.directive.ts 73.73% <100%> (ø) ⬆️
src/tooltip/tooltip.directive.ts 67.51% <100%> (-1.77%) ⬇️
src/chronos/public_api.ts 100% <100%> (ø) ⬆️
src/collapse/collapse.directive.ts 72.56% <25%> (-3.63%) ⬇️
src/chronos/i18n/kk.ts 66.66% <66.66%> (ø)
src/datepicker/bs-datepicker.component.ts 68.25% <81.81%> (+2.73%) ⬆️
src/typeahead/typeahead.directive.ts 79.92% <85.71%> (-1.21%) ⬇️
... and 22 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 a178962...1a0e5d3. Read the comment docs.

valorkin
valorkin previously approved these changes Feb 27, 2020
Copy link
Member

@valorkin valorkin left a comment

Choose a reason for hiding this comment

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

LGTM overall
@daniloff200 please create an issue to move daterange specific staff to range picker and remove from date picker

IraErshova
IraErshova previously approved these changes Mar 2, 2020
@daniloff200 daniloff200 merged commit b405057 into development Mar 2, 2020
@delete-merged-branch delete-merged-branch bot deleted the show-one-month-in-daterangepicker branch March 2, 2020 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change month position when maxDate is today
4 participants