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
fix: Add placement option to date/datetime picker #925
fix: Add placement option to date/datetime picker #925
Conversation
Deploy preview for fundamental-ngx ready! Built with commit 01be2ab |
@@ -19,7 +19,8 @@ <h2>Range Date Picker</h2> | |||
|
|||
<separator></separator> | |||
<h2>Internationalization</h2> | |||
<description>It is possible to internationalize both the aria labels and the month names/weekdays through providing a service.</description> | |||
<description>It is possible to internationalize both the aria labels and the month names/weekdays through providing a service. | |||
There is also added <code>[position]</code> attribute which allows us to decide where popup should be shown.</description> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this note probably does not belong in the "internationalisation" section
@@ -52,7 +52,7 @@ export class CustomI18nLabels extends CalendarI18nLabels { | |||
|
|||
@Component({ | |||
selector: 'fd-datepicker-i18n-example', | |||
template: `<fd-date-picker [(ngModel)]="selectedDay"></fd-date-picker>`, | |||
template: `<fd-date-picker [(ngModel)]="selectedDay" [placement]="'top-end'"></fd-date-picker>`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Again this probably does not belong in the i18n section but may warrant its own example
@JKMarkowski would you mind fixing the merge conflicts |
@mikerodonnell89 Sure, it's done. |
Please provide a link to the associated issue.
fixes #775
Please provide a brief summary of this pull request.
I added placement input to date and datetime picker.
If this is a new feature, have you updated the documentation?
I added examples on date and datetime docs.