-
Notifications
You must be signed in to change notification settings - Fork 45
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
fixed issue for Persian language #1162
Conversation
@@ -729,13 +729,18 @@ if (typeof window.DatePickerWidget === 'undefined') { | |||
} | |||
|
|||
#getLocalizedYear(date) { | |||
if(this.#lang === 'th') { |
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.
Why do you need a language specific check in the widget ?
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #1162 +/- ##
=========================================
Coverage 80.75% 80.75%
Complexity 773 773
=========================================
Files 91 91
Lines 2110 2110
Branches 285 285
=========================================
Hits 1704 1704
Misses 252 252
Partials 154 154 ☔ View full report in Codecov by Sentry. |
@@ -729,13 +729,18 @@ if (typeof window.DatePickerWidget === 'undefined') { | |||
} | |||
|
|||
#getLocalizedYear(date) { | |||
if(this.#lang === 'th') { |
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.
@devgurjar Add a comment saying that since thai locale has a different year, we need to do following
39cb811
to
f18e03c
Compare
f18e03c
to
1285fab
Compare
1285fab
to
275bac7
Compare
275bac7
to
aba758d
Compare
Lighthouse scores (desktop)
|
Lighthouse scores (mobile)
|
Accessibility Violations Found
|
2 similar comments
Accessibility Violations Found
|
Accessibility Violations Found
|
Co-authored-by: dgurjar <dgurjar@adobe.com>
Co-authored-by: dgurjar <dgurjar@adobe.com>
Description
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: