Skip to content

Conversation

MichaelJamesParsons
Copy link
Collaborator

@MichaelJamesParsons MichaelJamesParsons commented Oct 22, 2020

Refactor material datepicker so apps can bind custom picker components to date inputs.

API changes

  • Renamed the following in MatDatepickerBase:
    • _registerInput() -> registerInput()
    • _stateChanges -> stateChanges
    • _datepickerInput -> datepickerInput
  • Added MatDatepickerPanel interface (recommendations for another name?).
  • Refactored MatDatepickerInput, MatDateRangeInput, and MatDatepickerToggle to receive a MatDatepickerPanel instead of a specific implementation of MatDatepickerBase.

@google-cla google-cla bot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Oct 22, 2020
@MichaelJamesParsons MichaelJamesParsons force-pushed the datepicker-custom-picker-panels branch from 36773a0 to dc8c7bb Compare October 22, 2020 21:52
@MichaelJamesParsons MichaelJamesParsons added area: material/datepicker G This is is related to a Google internal issue P2 The issue is important to a large percentage of users, with a workaround labels Oct 22, 2020
@jelbourn jelbourn added the target: minor This PR is targeted for the next minor release label Oct 22, 2020
@MichaelJamesParsons MichaelJamesParsons force-pushed the datepicker-custom-picker-panels branch 4 times, most recently from a963660 to 75b1ce0 Compare October 23, 2020 03:19
Copy link
Member

@jelbourn jelbourn left a comment

Choose a reason for hiding this comment

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

One last comment from me, @mmalerba should also take a quick look

openedStream: EventEmitter<void>;
stateChanges: Subject<void>;
open(): void;
registerInput(input: C): MatDateSelectionModel<S, D>;
Copy link
Member

Choose a reason for hiding this comment

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

Since this is a public API, each of these members need a JsDoc description (these are auto generated into material.angular.io)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done

Copy link
Contributor

@mmalerba mmalerba left a comment

Choose a reason for hiding this comment

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

LGTM after Jeremy's comment is addressed

@MichaelJamesParsons MichaelJamesParsons force-pushed the datepicker-custom-picker-panels branch from 75b1ce0 to 1648eea Compare October 27, 2020 19:57
Copy link
Member

@jelbourn jelbourn left a comment

Choose a reason for hiding this comment

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

LGTM

@jelbourn jelbourn added the action: merge The PR is ready for merge by the caretaker label Oct 27, 2020
@annieyw annieyw merged commit 5b1660e into angular:master Nov 4, 2020
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Dec 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker area: material/datepicker cla: yes PR author has agreed to Google's Contributor License Agreement G This is is related to a Google internal issue P2 The issue is important to a large percentage of users, with a workaround target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants