Skip to content

Add support for min-date max-date for dateRange directive#299

Open
ramseyfeng wants to merge 1 commit intog00fy-:masterfrom
ramseyfeng:support_date_range_min_max
Open

Add support for min-date max-date for dateRange directive#299
ramseyfeng wants to merge 1 commit intog00fy-:masterfrom
ramseyfeng:support_date_range_min_max

Conversation

@ramseyfeng
Copy link
Copy Markdown

We used this directive in our project, it's better we can also define min & max value for the dateRange directive.

@ramseyfeng ramseyfeng changed the title Add support for min max for dateRange directive Add support for min-date max-date for dateRange directive Jul 27, 2016
@sihuiIE
Copy link
Copy Markdown

sihuiIE commented Aug 19, 2016

Thank you for the update,

i have the same issue,
but my html set is not working with this new update .
Am I missing some thing?

here is my html

<html>
    <head>
        <title>sample</title>
        <meta charset="utf-8">
        <meta http-equiv="x-ua-compatible" content="ie=edge">
        <title></title>
        <meta name="description" content="">
        <meta name="viewport" content="width=device-width, initial-scale=1">
        <link rel="apple-touch-icon" href="apple-touch-icon.png">
    </head>

    <body id="body" ng-app="sample">
        <form  class="test-unit">
            <input type="datetime" date-time ng-model="demo" min-view="date" min-date="2016-08-20" max-date="2016-09-20" max-view="date" auto-close="true" view="date" maxlength="10" placeholder="Set Date" format="DD/MM/YYYY">
        </form>

        <script src="https://cdnjs.cloudflare.com/ajax/libs/angular.js/1.5.8/angular.min.js"></script>
        <script src="https://cdnjs.cloudflare.com/ajax/libs/moment.js/2.10.6/moment.min.js"></script>
        <script src="https://rawgit.com/g00fy-/angular-datepicker/master/dist/angular-datepicker.min.js"></script>
        <script src="./dateRange.js"></script>
        <script>
        var demoApp = angular.module('sample', ['datePicker']);
        </script>
    </body>
</html>

could you give me a hit?

@ramseyfeng
Copy link
Copy Markdown
Author

ramseyfeng commented Aug 26, 2016

This pull request has not been merged to branch yet.
Do you have a plunker for it.

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.

2 participants