Skip to content
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

Added DD-MM-YYYY format option button in configuration menu. #3710

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

YashKale313
Copy link

Added DD-MM-YYYY format option button in configuration menu.

@mmahmoudian
Copy link
Member

Thanks. Since your proposal was to have even number of options, would you please also add the century (or something similar).

@YashKale313
Copy link
Author

@mmahmoudian Actually the logic that add format buttons ensures that there are even number of formats passed. So before threre were 21 formats (including century) and it was only adding 20 formats excluding century but now since i added 1 it has become 22 formats and its showing all 22 as its even.

@YashKale313
Copy link
Author

YashKale313 commented Sep 5, 2024

@mmahmoudian Here are the code snippets for your refrence

For adding formats-
image

For adding buttons-
image

let me know if you feels anything wrong here.

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