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

feat: Greek Translations, Create el.json #3836

Merged
merged 15 commits into from
Dec 15, 2023
Merged

Conversation

sl45sms
Copy link
Contributor

@sl45sms sl45sms commented Dec 19, 2022

Greek Translation (el)

Greek Translation
@sl45sms sl45sms requested a review from a team as a code owner December 19, 2022 09:57
@CLAassistant
Copy link

CLAassistant commented Dec 19, 2022

CLA assistant check
All committers have signed the CLA.

@WilcoFiers
Copy link
Contributor

@sl45sms Wow thank you! Apologies for the delayed response. This one slipped through in the vacation. I'll need to find someone who can help us review this, which from past experience can take a while. So heads up on that.

@sl45sms sl45sms changed the title Create el.json feat: Greek Translations, Create el.json Jan 20, 2023
fix space on ${data. fontWeight}
@WilcoFiers
Copy link
Contributor

Just another quick ping to let you know we haven't forgotten. As I warned this can be slow, but I'm still working on it!

@sl45sms
Copy link
Contributor Author

sl45sms commented Feb 21, 2023

Just another quick ping to let you know we haven't forgotten. As I warned this can be slow, but I'm still working on it!

Great, not rush, take the time as needed, thanks a lot.

Copy link
Contributor

@straker straker left a comment

Choose a reason for hiding this comment

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

Apologies for the long delay in getting this reviewed. We're hoping to pull it into the 4.8 release happening soon. Thanks again for taking the time to translate.

Below are a few questions and suggestions. Importantly some of the wording has changed since you made this translation and I wanted to point that out to see if you wanted to update the wording to reflect the current values.

locales/el.json Outdated
"help": "Τα ενεργά στοιχεία <area> πρέπει να έχουν εναλλακτικό κείμενο"
},
"aria-allowed-attr": {
"description": "Διασφαλίζει ότι τα χαρακτηριστικά ARIA επιτρέπονται για το ρόλο του στοιχείου",
Copy link
Contributor

Choose a reason for hiding this comment

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

We've recently moved some things around and changed the wording of aria-allowed-attr from allowed to supports. If we make the change now it'll make sure someone doesn't miss updating the text change for the next time.

Here's the new text:

Ensures an element's role supports its ARIA attributes

Copy link
Contributor Author

Choose a reason for hiding this comment

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

changed

Choose a reason for hiding this comment

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

Thanks

locales/el.json Outdated
"help": "Ο ρόλος ARIA θα πρέπει να είναι κατάλληλος για το στοιχείο"
},
"aria-command-name": {
"description": "Διασφαλίζει ότι κάθε ARIA σε κουμπί,σύνδεσμο ή στοιχείο μενού έχει ένα προσβάσιμο όνομα",
Copy link
Contributor

Choose a reason for hiding this comment

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

button, link, and menuitem are specific names of ARIA role values. Other languages typically keep these translated as English words wrapped in quotes or brackets. Would it be helpful to do the same in Greek?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it is clear for the Greek programmers, but for compatibility reasons I will add them.

locales/el.json Outdated
},
"color-contrast-enhanced": {
"description": "Διασφαλίζει ότι η αντίθεση μεταξύ των χρωμάτων προσκηνίου και φόντου πληροί τα όρια αναλογίας αντίθεσης WCAG 2 AAA",
"help": "Τα στοιχεία πρέπει να έχουν επαρκή χρωματική αντίθεση"
Copy link
Contributor

Choose a reason for hiding this comment

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

The color-contrast-enhanced help text should be different than the color-contrast help text so they can be distinguished, especially when used in the extension.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i will do that.

locales/el.json Outdated
"help": "Οι εικόνες πρέπει να έχουν εναλλακτικό κείμενο"
},
"image-redundant-alt": {
"description": "Διασφαλίζει ότι η εναλλακτική εικόνα δεν επαναλαμβάνεται ως κείμενο",
Copy link
Contributor

Choose a reason for hiding this comment

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

This deals more with the images alt text being redundant more than an alternative image. Should it use εναλλακτικό κείμενο from the image-alt description rather than εναλλακτική εικόνα?

locales/el.json Outdated
}
},
"aria-hidden-body": {
"pass": "Δεν υπάρχει χαρακτηριστικό arria-hidden στο σώμα του εγγράφου",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"pass": "Δεν υπάρχει χαρακτηριστικό arria-hidden στο σώμα του εγγράφου",
"pass": "Δεν υπάρχει χαρακτηριστικό aria-hidden στο σώμα του εγγράφου",

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

locales/el.json Outdated
"fail": {
"singular": "Ο απαιτούμενος θυγατρικός ρόλος ARIA δεν υπάρχει: ${data.values}",
"plural": "Οι απαιτούμενοι θυγατρικόι ρόλοι ARIA δεν υπάρχουν: ${data.values}",
"unallowed": "Το στοιχείο έχει γόνους που δεν επιτρέπονται (βλ. σχετικούς κόμβους)"
Copy link
Contributor

Choose a reason for hiding this comment

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

Another minor update we made since this was translated.

Suggested change
"unallowed": "Το στοιχείο έχει γόνους που δεν επιτρέπονται (βλ. σχετικούς κόμβους)"
"unallowed": "Το στοιχείο έχει γόνους που δεν επιτρέπονται: ${data.values}"

locales/el.json Outdated
"pass": "Το στοιχείο έχει επαρκή χρωματική αντίθεση ${data.contrastRatio}",
"fail": {
"default": "Το στοιχείο έχει ανεπαρκή χρωματική αντίθεση ${data.contrastRatio} (χρώμα προσκηνίου: ${data.fgColor}, χρώμα φόντου: ${data.bgColor}, μέγεθος γραμματοσειράς: ${data.fontSize}, βάρος γραμματοσειράς: ${data.fontWeight}). Αναμενόμενος λόγος αντίθεσης ${data.expectedContrastRatio}",
"fgOnShadowColor": "Το στοιχείο έχει ανεπαρκή χρωματική αντίθεση ${data.contrastRatio} μεταξύ του χρώματος του προσκηνίου και του χρώματος σκιάς (χρώμα προσκηνίου: ${data.fgColor}, χρώμα σκιάς κειμένου: ${data.shadowColor}, μέγεθος γραμματοσειράς: ${data.fontSize} , βάρος γραμματοσειράς: ${data.fontWeight}). Αναμενόμενος λόγος αντίθεσης ${data.expectedContrastRatio}",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"fgOnShadowColor": "Το στοιχείο έχει ανεπαρκή χρωματική αντίθεση ${data.contrastRatio} μεταξύ του χρώματος του προσκηνίου και του χρώματος σκιάς (χρώμα προσκηνίου: ${data.fgColor}, χρώμα σκιάς κειμένου: ${data.shadowColor}, μέγεθος γραμματοσειράς: ${data.fontSize} , βάρος γραμματοσειράς: ${data.fontWeight}). Αναμενόμενος λόγος αντίθεσης ${data.expectedContrastRatio}",
"fgOnShadowColor": "Το στοιχείο έχει ανεπαρκή χρωματική αντίθεση ${data.contrastRatio} μεταξύ του χρώματος του προσκηνίου και του χρώματος σκιάς (χρώμα προσκηνίου: ${data.fgColor}, χρώμα σκιάς κειμένου: ${data.shadowColor}, μέγεθος γραμματοσειράς: ${data.fontSize}, βάρος γραμματοσειράς: ${data.fontWeight}). Αναμενόμενος λόγος αντίθεσης ${data.expectedContrastRatio}",

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

locales/el.json Outdated
},
"fail": {
"default": "Το στοιχείο έχει ανεπαρκή χρωματική αντίθεση ${data.contrastRatio} (χρώμα προσκηνίου: ${data.fgColor}, χρώμα φόντου: ${data.bgColor}, μέγεθος γραμματοσειράς: ${data.fontSize}, βάρος γραμματοσειράς: ${data.fontWeight}). Αναμενόμενος λόγος αντίθεσης ${data.expectedContrastRatio}",
"fgOnShadowColor": "Το στοιχείο έχει ανεπαρκή χρωματική αντίθεση ${data.contrastRatio} μεταξύ του χρώματος προσκηνίου και σκιάς (χρώμα προσκηνίου: ${data.fgColor}, χρώμα σκιάς κειμένου: ${data.shadowColor}, μέγεθος γραμματοσειράς: ${data.fontSize} , βάρος γραμματοσειράς: ${data.fontWeight}). Αναμενόμενος λόγος αντίθεσης ${data.expectedContrastRatio}",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"fgOnShadowColor": "Το στοιχείο έχει ανεπαρκή χρωματική αντίθεση ${data.contrastRatio} μεταξύ του χρώματος προσκηνίου και σκιάς (χρώμα προσκηνίου: ${data.fgColor}, χρώμα σκιάς κειμένου: ${data.shadowColor}, μέγεθος γραμματοσειράς: ${data.fontSize} , βάρος γραμματοσειράς: ${data.fontWeight}). Αναμενόμενος λόγος αντίθεσης ${data.expectedContrastRatio}",
"fgOnShadowColor": "Το στοιχείο έχει ανεπαρκή χρωματική αντίθεση ${data.contrastRatio} μεταξύ του χρώματος προσκηνίου και σκιάς (χρώμα προσκηνίου: ${data.fgColor}, χρώμα σκιάς κειμένου: ${data.shadowColor}, μέγεθος γραμματοσειράς: ${data.fontSize}, βάρος γραμματοσειράς: ${data.fontWeight}). Αναμενόμενος λόγος αντίθεσης ${data.expectedContrastRatio}",

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

"pass": "Η ετικέτα <meta> δεν απενεργοποιεί το ζουμ σε κινητές συσκευές",
"fail": "Η ετικέτα ${data} στην ετικέτα <meta> απενεργοποιεί το ζουμ σε κινητές συσκευές"
},
"target-offset": {
Copy link
Contributor

Choose a reason for hiding this comment

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

The target-offset wording has been changed to use space instead of offset and use terms associated with distance of a circle as the spec was updated to use the distance between two circles to determine the spacing.

Here's a link to the new wording: https://github.com/dequelabs/axe-core/blob/develop/locales/_template.json#L864-L870

Copy link
Contributor Author

Choose a reason for hiding this comment

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

changed

locales/el.json Outdated
"default": "Το στοιχείο με αρνητικό tabindex έχει ανεπαρκές μέγεθος (${data.width}px επί ${data.height}px, θα πρέπει να είναι τουλάχιστον ${data.minSize}px επί ${data.minSize}px). Είναι στόχος;",
"contentOverflow": "Το μέγεθος του στοιχείου δεν ήταν δυνατό να προσδιοριστεί με ακρίβεια λόγω του περιεχομένου υπερχείλισης",
"partiallyObscured": "Το στοιχείο με αρνητικό tabindex δεν έχει επαρκές μέγεθος επειδή είναι μερικώς κρυμμένο (το μικρότερο διάστημα είναι ${data.width}px επί ${data.height}px, θα πρέπει να είναι τουλάχιστον ${data.minSize}px επί ${data.minSize} px). Είναι στόχος;",
"partiallyObscuredNonTabbable": "Ο στόχος έχει ανεπαρκές μέγεθος επειδή είναι μερικώς καλυμμένος από έναν γείτονα με αρνητικό tabindex (το μικρότερο διάστημα είναι ${data.width}px επί ${data.height}px, θα πρέπει να είναι τουλάχιστον ${data.minSize}px επί ${data .minSize}px). Είναι στόχος ο γείτονας;"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"partiallyObscuredNonTabbable": "Ο στόχος έχει ανεπαρκές μέγεθος επειδή είναι μερικώς καλυμμένος από έναν γείτονα με αρνητικό tabindex (το μικρότερο διάστημα είναι ${data.width}px επί ${data.height}px, θα πρέπει να είναι τουλάχιστον ${data.minSize}px επί ${data .minSize}px). Είναι στόχος ο γείτονας;"
"partiallyObscuredNonTabbable": "Ο στόχος έχει ανεπαρκές μέγεθος επειδή είναι μερικώς καλυμμένος από έναν γείτονα με αρνητικό tabindex (το μικρότερο διάστημα είναι ${data.width}px επί ${data.height}px, θα πρέπει να είναι τουλάχιστον ${data.minSize}px επί ${data.minSize}px). Είναι στόχος ο γείτονας;"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

straker and others added 13 commits August 29, 2023 16:11
change aria-allowed-attr to support
ARIA role values
distinguished contrast
fix image-redundant-alt
add values on unallowed
remove space
remove space
fix target-offset
remove space
fix aria-allowed-attr
@straker
Copy link
Contributor

straker commented Sep 25, 2023

@sl45sms Am I correct to assume that this PR is ready for review again with all your updates?

@sl45sms
Copy link
Contributor Author

sl45sms commented Sep 25, 2023

@sl45sms Am I correct to assume that this PR is ready for review again with all your updates?

Yes please, review the updates. Thanks.

@straker
Copy link
Contributor

straker commented Nov 28, 2023

Thanks for the translations! Approved for security

@straker straker merged commit 3ea9a48 into dequelabs:develop Dec 15, 2023
1 check was pending
WilcoFiers added a commit that referenced this pull request Mar 25, 2024
##
[4.9.0](v4.8.4...v4.9.0)
(2024-03-25)

### Features

- adding the wcag131 tag to the aria-hidden-body rule
([#4349](#4349))
([dd4c3c3](dd4c3c3)),
closes [#4315](#4315)
- **checks:** deprecate aria-busy check
([#4356](#4356))
([be0b555](be0b555)),
closes [#4347](#4347)
[#4340](#4340)
- **color:** add color channel values and luminosity, saturation, clip
functions ([#4366](#4366))
([9e70199](9e70199)),
closes
[/github.com//pull/4365/files#r1517706612](https://github.com/dequelabs//github.com/dequelabs/axe-core/pull/4365/files/issues/r1517706612)
- **i18n:** add Greek Translations
([#3836](#3836))
([3ea9a48](3ea9a48))
- **i18n:** Add Italian translation
([#4344](#4344))
([de1baa9](de1baa9))
- **i18n:** Add Simplified Chinese translation
([#4379](#4379))
([bda7c8d](bda7c8d))
- **i18n:** Add Taiwanese Mandarin translation
([#4299](#4299))
([c5e11de](c5e11de))

### Bug Fixes

- Add LICENSE-3RD-PARTY.txt file
([#4304](#4304))
([daa0fe6](daa0fe6))
- add Object.values polyfill for node <=6
([#4274](#4274))
([5eb867b](5eb867b))
- **aria-required-children:** avoid confusing aria-busy message in
failures ([#4347](#4347))
([591607d](591607d)),
closes [#fail13](https://github.com/dequelabs/axe-core/issues/fail13)
[#4340](#4340)
- avoid reading element-specific node properties of non-element node
types ([#4317](#4317))
([b853b18](b853b18)),
closes [#4316](#4316)
[#4316](#4316)
- **color-contrast:** handle text that is outside `overflow: hidden`
ancestor ([#4357](#4357))
([bdb7300](bdb7300)),
closes [#4253](#4253)
- **color-contrast:** support color blend modes hue, saturation, color,
luminosity ([#4365](#4365))
([7ae4761](7ae4761))
- **d.ts:** RawNodesResult issues
([#4229](#4229))
([d660518](d660518))
- **d.ts:** RunOptions.reporter can be any string
([#4218](#4218))
([e53f5c5](e53f5c5))
- **i18n:** update Italian translations
([#4377](#4377))
([4d65d4b](4d65d4b))
- **listitem:** clarify roleNotValid message
([#4374](#4374))
([0f8a9af](0f8a9af))
- **scrollable-region-focusable:** missing wcag213 tag
([#4201](#4201))
([0080a72](0080a72))
- **target-size:** always pass 10x targets (avoid perf bottleneck)
([#4376](#4376))
([be327c4](be327c4))
- **target-size:** do not crash for nodes with many overlapping widgets
([#4373](#4373))
([1dbea83](1dbea83)),
closes [#4359](#4359)
[#4359](#4359)
[#4360](#4360)
- **utils/get-selector:** ignore 'xmlns' attribute when generating a
selector ([#4303](#4303))
([938b411](938b411))

This PR was opened by a robot 🤖 🎉
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.

5 participants