Skip to content

Commit

Permalink
Maintenance: Improved translatable strings.
Browse files Browse the repository at this point in the history
Co-authored-by: Martin Gruner <mg@zammad.com>
Co-authored-by: Dusan Vuckovic <dv@zammad.com>
  • Loading branch information
mgruner and dvuckovic committed Jan 25, 2024
1 parent 443a4c5 commit db33c5c
Show file tree
Hide file tree
Showing 5 changed files with 17 additions and 11 deletions.
Expand Up @@ -332,7 +332,7 @@ const duration = VITE_TEST_MODE ? undefined : { enter: 300, leave: 200 }
@keypress.enter.prevent.stop="selectAll()"
@keypress.space.prevent.stop="selectAll()"
>
{{ $t('select all') }}
{{ $t('select all options') }}
</CommonLabel>
</div>
<div
Expand Down
Expand Up @@ -739,7 +739,7 @@ describe('Form - Field - Select - Features', () => {
const menu = wrapper.getByRole('menu')

let selectAllButton = getByRole(menu, 'button', {
name: 'select all',
name: 'select all options',
})

const listbox = getByRole(menu, 'listbox')
Expand Down Expand Up @@ -826,7 +826,7 @@ describe('Form - Field - Select - Features', () => {
})

selectAllButton = getByRole(menu, 'button', {
name: 'select all',
name: 'select all options',
})

expect(selectAllButton).toBeInTheDocument()
Expand Down Expand Up @@ -1138,7 +1138,9 @@ describe('Form - Field - Select - Accessibility', () => {

const menu = wrapper.getByRole('menu')

const selectAllButton = getByRole(menu, 'button', { name: 'select all' })
const selectAllButton = getByRole(menu, 'button', {
name: 'select all options',
})

expect(selectAllButton).toHaveAttribute('tabindex', '1')

Expand Down
Expand Up @@ -459,7 +459,9 @@ const duration = VITE_TEST_MODE ? undefined : { enter: 300, leave: 200 }
@keypress.space.prevent.stop="selectAll()"
>
{{
currentPath.length ? $t('select visible') : $t('select all')
currentPath.length
? $t('select visible options')
: $t('select all options')
}}
</CommonLabel>
</div>
Expand Down
Expand Up @@ -664,7 +664,7 @@ describe('Form - Field - TreeSelect - Features', () => {
const menu = wrapper.getByRole('menu')

const selectAllButton = getByRole(menu, 'button', {
name: 'select all',
name: 'select all options',
})

const listbox = getByRole(menu, 'listbox')
Expand Down Expand Up @@ -774,7 +774,7 @@ describe('Form - Field - TreeSelect - Features', () => {

expect(
queryByRole(menu, 'button', {
name: 'select visible',
name: 'select visible options',
}),
).not.toBeInTheDocument()

Expand All @@ -791,7 +791,7 @@ describe('Form - Field - TreeSelect - Features', () => {
})

const selectVisibleButton = getByRole(menu, 'button', {
name: 'select visible',
name: 'select visible options',
})

await wrapper.events.click(selectVisibleButton)
Expand Down Expand Up @@ -1256,7 +1256,9 @@ describe('Form - Field - TreeSelect - Accessibility', () => {

const menu = wrapper.getByRole('menu')

const selectAllButton = getByRole(menu, 'button', { name: 'select all' })
const selectAllButton = getByRole(menu, 'button', {
name: 'select all options',
})

expect(selectAllButton).toHaveAttribute('tabindex', '2')

Expand Down
4 changes: 2 additions & 2 deletions i18n/zammad.pot
Expand Up @@ -15889,7 +15889,7 @@ msgstr ""

#: app/frontend/apps/desktop/components/CommonSelect/CommonSelect.vue
#: app/frontend/apps/desktop/components/Form/fields/FieldTreeSelect/FieldTreeSelectInputDropdown.vue
msgid "select all"
msgid "select all options"
msgstr ""

#: app/assets/javascripts/app/views/generic/attachment.jst.eco
Expand All @@ -15898,7 +15898,7 @@ msgid "select attachment…"
msgstr ""

#: app/frontend/apps/desktop/components/Form/fields/FieldTreeSelect/FieldTreeSelectInputDropdown.vue
msgid "select visible"
msgid "select visible options"
msgstr ""

#: app/assets/javascripts/app/controllers/_ui_element/core_workflow_perform.coffee
Expand Down

0 comments on commit db33c5c

Please sign in to comment.