Skip to content

sap.ui.unified.Calendar$SelectEventParameters should have the selected date #4245

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

Closed
daletman opened this issue Mar 25, 2025 · 4 comments
Closed

Comments

@daletman
Copy link

Why are you proposing this feature?

sap.ui.unified.Calendar$SelectEventParameters should have the selected date

How should a possible solution look like?

No response

Are there alternative approaches?

No response

Any further information you would like to share?

No response

@s-todorova s-todorova self-assigned this Mar 25, 2025
@s-todorova
Copy link
Contributor

Hi @daletman,

Thanks for the feature request, I just want to make sure I fully understand the need behind this. Based on your description, are you looking to access the selected date directly from the event parameters of the select event?

Currently, the sample CalendarSingleDaySelection demonstrates how to extract the selected date.

The Calendar control keeps the selected dates, which can be queried from the event source (oEvent.getSource()) after the selection.

Could you please clarify your use case and why you think a parameter should be introduced?
Understanding your reasoning will help us evaluate the request better and decide whether this is a feasible request, since a way to access the selected date already exists and is documented.

Best reagrds,
Siyana

@daletman
Copy link
Author

You're right, that there is a way to get the selected day, but isn't the whole point of event parameters to directly get the obviously relevant information? You can get a lot of other event parameters of other controls without them being directly in the event object, but they are still there.

I basically was just expecting it to be there based on my experience with other event objects in SAPUI5.

@s-todorova
Copy link
Contributor

Hi @daletman,

I've created an internal incident DINC0452910. The status of the issue will be updated here in GitHub.

Regards,
Siyana

@tsanislavgatev
Copy link
Contributor

Hi @daletman ,

I don't agree on introducing another event parameter for something that is already updated in the control and it is available to get quite easily. The idea behind this event is to notify that change in selection has occured and then the application developer can get the selection to receive the latest state. If there is an information needed that is not updated or easy to access we can evaluate and if needed we can add thet information to the details, but as for now I don't see the need for such details in the event.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants