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

added logic for opening datepicker calendar #249

Merged
merged 1 commit into from
Mar 16, 2017
Merged

Conversation

kuzma-sasha
Copy link

when datepicker calendar was opened, the month of the selected date or of the current date is displayed

when datepicker calendar was opened, the month of the selected date or current date is displayed
@45kb
Copy link
Member

45kb commented Mar 16, 2017

@kuzma-sasha thanks for this, can you describe what problem are you fixing and how to replicate it?

Because if i select a date or a day then when i close and reopen the datepicker i see it's already selected...

@kuzma-sasha
Copy link
Author

Hello) There is the following situation:

  1. open the calendar;
  2. choose the date from the calendar - March 10, 2017 (03-10-2017);
  3. close the calendar;
  4. open the calendar and we can see selected date and selected month, it is right;
  5. change date manually with other month, for example February 11, 2017 (02-11-2017);
  6. close the calendar (using focus out);
  7. open the calendar;
  8. result: we can see calendar with March month, and we don's see selected day;

@45kb
Copy link
Member

45kb commented Mar 16, 2017

works great @kuzma-sasha thanks, going to release it!

@45kb 45kb merged commit 4525a7e into 720kb:master Mar 16, 2017
45kb added a commit that referenced this pull request Mar 16, 2017
boltmedia pushed a commit to contentco/angular-datepicker that referenced this pull request Apr 2, 2017
* commit '8f882739dcb2e1b07c42cb6ae1f4e2872fe09354':
  #251 fix thanks @PizzaPete
  fix by @PizzaPete see #251
  Update README.md
  #251 - Changed the item into the $index.
  Fix markdown headings in the README
  720kb#249 thanks @kuzma-sasha

# Conflicts:
#	dist/angular-datepicker.min.js
#	dist/angular-datepicker.sourcemap.map
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.

2 participants