{"action":"opened","issue":{"url":"https://api.github.com/repos/ng-bootstrap/ng-bootstrap/issues/2913","repository_url":"https://api.github.com/repos/ng-bootstrap/ng-bootstrap","labels_url":"https://api.github.com/repos/ng-bootstrap/ng-bootstrap/issues/2913/labels{/name}","comments_url":"https://api.github.com/repos/ng-bootstrap/ng-bootstrap/issues/2913/comments","events_url":"https://api.github.com/repos/ng-bootstrap/ng-bootstrap/issues/2913/events","html_url":"https://github.com/ng-bootstrap/ng-bootstrap/issues/2913","id":388656646,"node_id":"MDU6SXNzdWUzODg2NTY2NDY=","number":2913,"title":"Allow markDisabled to be run after 'navigate'","user":{"login":"CarlosTorrecillas","id":648793,"node_id":"MDQ6VXNlcjY0ODc5Mw==","avatar_url":"https://avatars3.githubusercontent.com/u/648793?v=4","gravatar_id":"","url":"https://api.github.com/users/CarlosTorrecillas","html_url":"https://github.com/CarlosTorrecillas","followers_url":"https://api.github.com/users/CarlosTorrecillas/followers","following_url":"https://api.github.com/users/CarlosTorrecillas/following{/other_user}","gists_url":"https://api.github.com/users/CarlosTorrecillas/gists{/gist_id}","starred_url":"https://api.github.com/users/CarlosTorrecillas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/CarlosTorrecillas/subscriptions","organizations_url":"https://api.github.com/users/CarlosTorrecillas/orgs","repos_url":"https://api.github.com/users/CarlosTorrecillas/repos","events_url":"https://api.github.com/users/CarlosTorrecillas/events{/privacy}","received_events_url":"https://api.github.com/users/CarlosTorrecillas/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-12-07T13:33:16Z","updated_at":"2018-12-07T13:33:16Z","closed_at":null,"author_association":"NONE","body":"### Bug description:\r\n\r\n`markDisabled `is executed before the `navigate `callback in NgbDatePicker when changing month. If you want to apply some checks re the state of the days in terms of them being disabled then that's not possible. \r\n\r\n### Link to minimally-working StackBlitz that reproduces the issue:\r\n\r\nhttps://stackblitz.com/edit/datepicker-navigation\r\n\r\n### Versions of Angular, ng-bootstrap and Bootstrap:\r\n\r\nAngular: 7.0.0\r\n\r\nng-bootstrap: 4.0.0\r\n\r\nBootstrap: -\r\n\r\nI think this is pretty similar to https://github.com/ng-bootstrap/ng-bootstrap/issues/1325 and I could also use the workaround but in my case and in general we could make that event being run after the navigate.\r\n\r\nI'm not sure if that would be a matter of reordering the lists:\r\n\r\nhttps://github.com/ng-bootstrap/ng-bootstrap/blob/master/src/datepicker/datepicker.ts#L281\r\nhttps://github.com/ng-bootstrap/ng-bootstrap/blob/master/src/datepicker/datepicker.ts#L289\r\n\r\nAnd if you don't see any issues with that perhaps that could be a feature to have?"}}
{"actor":{"display_login":"CarlosTorrecillas"}}