Skip to content

Commit

Permalink
Merge pull request #5130 from nextcloud-libraries/fix/a11y/41842/NcTi…
Browse files Browse the repository at this point in the history
…mezonePicker-group-headings-do-not-meet-contrast-ratio
  • Loading branch information
skjnldsv authored Jan 25, 2024
2 parents 145e832 + b4e6166 commit 7fdd4e0
Showing 1 changed file with 8 additions and 5 deletions.
13 changes: 8 additions & 5 deletions src/components/NcTimezonePicker/NcTimezonePicker.vue
Original file line number Diff line number Diff line change
Expand Up @@ -115,15 +115,18 @@ export default {
/**
* Since NcSelect does not support groups,
* we create an object with the grouped timezones and continent labels.
*
* NOTE for now we are removing the grouping from the fields to fix an accessibility issue
* in the future, other options can be introduced to better display the different areas
*/
let timezonesGrouped = []
Object.values(timezoneList).forEach(group => {
// Add an entry as group label
timezonesGrouped.push({
label: group.continent,
timezoneId: `tz-group__${group.continent}`,
regions: group.regions,
})
// timezonesGrouped.push({
// label: group.continent,
// timezoneId: `tz-group__${group.continent}`,
// regions: group.regions,
// })
timezonesGrouped = timezonesGrouped.concat(group.regions)
})
return timezonesGrouped
Expand Down

0 comments on commit 7fdd4e0

Please sign in to comment.