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

12175 move texts configs into separate section #12238

Merged
merged 27 commits into from
Feb 12, 2024
Merged
Show file tree
Hide file tree
Changes from 20 commits
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
b2bc769
Move text-config to separate section
Feb 1, 2024
f9d20ff
Add texts for textResourceBindings on group-component
Feb 1, 2024
d3eeb84
Update tests
Feb 1, 2024
1551eed
Render all available textResourceBindings from schema by default
Feb 1, 2024
ada091a
Adapt container editing limitations information
Feb 2, 2024
0902154
Update tests
Feb 2, 2024
2130258
Remove texts from static component config
Feb 2, 2024
8c07664
add options section to texts accordion & update tests
nkylstad Feb 5, 2024
29a3fce
update layoutMock
nkylstad Feb 5, 2024
52b8294
add back text needed for v3
nkylstad Feb 5, 2024
12324e2
cleanup and update tests
nkylstad Feb 5, 2024
257beaa
Merge branch 'main' into 12175-move-texts-configs-into-separate-section
nkylstad Feb 5, 2024
b626544
Merge branch 'main' into 12175-move-texts-configs-into-separate-section
nkylstad Feb 6, 2024
d5beeaa
refactor EditTextResourceBindings and small cleanup
nkylstad Feb 6, 2024
51f026b
remove unneccessary code and add test
nkylstad Feb 6, 2024
c2bbc2c
more tests
nkylstad Feb 6, 2024
1d4b649
Merge branch 'main' into 12175-move-texts-configs-into-separate-section
nkylstad Feb 6, 2024
afa429e
Merge branch 'main' into 12175-move-texts-configs-into-separate-section
nkylstad Feb 6, 2024
e2d5235
Merge branch 'main' into 12175-move-texts-configs-into-separate-section
nkylstad Feb 7, 2024
5fa8b88
Merge branch 'main' into 12175-move-texts-configs-into-separate-section
nkylstad Feb 7, 2024
a0ed0d8
pr comments
nkylstad Feb 7, 2024
b1ece36
Merge branch 'main' into 12175-move-texts-configs-into-separate-section
nkylstad Feb 7, 2024
5a9ee37
updates from pr comments
nkylstad Feb 9, 2024
1b47775
Merge branch 'main' into 12175-move-texts-configs-into-separate-section
nkylstad Feb 9, 2024
82349f0
cleanup after merge
nkylstad Feb 9, 2024
552c9df
Merge remote-tracking branch 'origin/main' into 12175-move-texts-conf…
nkylstad Feb 12, 2024
53eb35e
ran prettier
nkylstad Feb 12, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 22 additions & 3 deletions frontend/language/src/nb.json
Original file line number Diff line number Diff line change
Expand Up @@ -1032,6 +1032,7 @@
"right_menu.rules_conditional_rendering_add_alt": "Legg til regel for vis/skjul felt",
"right_menu.rules_empty": "Ingen regler lagt til...",
"right_menu.show_old_dynamics": "Vis gammelt verktøy for dynamikk",
"right_menu.text": "Tekst",
"right_menu.warning_dynamics_deprecated": "Denne funksjonaliteten utgår og vi kommer ikke til å vedlikeholde den. Vi anbefaler å bruke <0 href=\"{{expressionDocs}}\" >dynamiske uttrykk.</0>",
"schema_editor.active_languages": "Aktive språk:",
"schema_editor.add": "Legg til",
Expand Down Expand Up @@ -1474,7 +1475,7 @@
"ux_editor.component_unknown": "Ukjent komponent",
"ux_editor.conditional_rendering_connection_header": "Betingede renderingstilkoblinger",
"ux_editor.container_empty": "Tomt, dra noe inn her...",
"ux_editor.container_not_editable_info": "Noen egenskaper for denne komponenten er ikke redigerbare for øyeblikket. Du kan legge til underkomponenter i kolonnen til venstre.",
"ux_editor.container_not_editable_info": "Noen egenskaper for denne komponenten er ikke redigerbare for øyeblikket. Du kan legge til underkomponenter i kolonnen til venstre og redigere tekster.",
"ux_editor.edit_component.show_beta_func": "Vis ny konfigurasjon (BETA)",
"ux_editor.edit_component.show_beta_func_help_text": "Vi jobber med å få på plass støtte for å redigere alle innstillinger. Ved å huke av her kan du ta i bruk den nye konfigurasjonsvisningen, som støtter flere innstillinger. Merk at denne visningen fortsatt er under utvikling, og vil kunne oppleves som noe ustabil.",
"ux_editor.edit_component.unknown_component": "Komponenten {{componentName}} gjenkjennes ikke av Studio og kan derfor ikke konfigureres.",
Expand Down Expand Up @@ -1541,6 +1542,7 @@
"ux_editor.modal_new_option": "Legg til flere",
"ux_editor.modal_options": "Valg",
"ux_editor.modal_properties_add_check_box_options": "Hvordan vil du legge til avkrysningsbokser?",
"ux_editor.modal_properties_add_options": "Hvordan vil du legge til alternativer?",
"ux_editor.modal_properties_add_radio_button_options": "Hvordan vil du legge til radioknapper?",
"ux_editor.modal_properties_button_helper": "Søk etter tekst til knappen",
"ux_editor.modal_properties_button_type_ActionButton": "ActionButton",
Expand Down Expand Up @@ -1609,6 +1611,8 @@
"ux_editor.modal_properties_read_only": "Read-only.",
"ux_editor.modal_properties_read_only_description": "Om dette feltet ikke skal kunne redigeres, kan du skru på read-only.",
"ux_editor.modal_properties_tag_helper": "Søk etter merkebeskrivelse",
"ux_editor.modal_properties_textResourceBindings_add_button": "'Legg til ny'-knapp",
"ux_editor.modal_properties_textResourceBindings_add_button_add": "Legg til tekst for 'Legg til ny'-knapp",
"ux_editor.modal_properties_textResourceBindings_altTextImg": "Alternativ tekst for bilde",
"ux_editor.modal_properties_textResourceBindings_altTextImg_add": "Legg til alternativ tekst for bilde",
"ux_editor.modal_properties_textResourceBindings_back": "Tekst på tilbake-knapp",
Expand All @@ -1617,16 +1621,24 @@
"ux_editor.modal_properties_textResourceBindings_body_add": "Legg til tekstinnhold",
"ux_editor.modal_properties_textResourceBindings_description": "Beskrivelse",
"ux_editor.modal_properties_textResourceBindings_description_add": "Legg til beskrivelse",
"ux_editor.modal_properties_textResourceBindings_edit_button_close": "'Rediger'-knapp (lukket gruppe)",
"ux_editor.modal_properties_textResourceBindings_edit_button_close_add": "Legg til tekst for 'Rediger'-knapp (lukket gruppe)",
"ux_editor.modal_properties_textResourceBindings_edit_button_open": "'Rediger'-knapp (åpen gruppe)",
"ux_editor.modal_properties_textResourceBindings_edit_button_open_add": "Legg til tekst for 'Rediger'-knapp (åpen gruppe)",
"ux_editor.modal_properties_textResourceBindings_help": "Hjelpetekst",
"ux_editor.modal_properties_textResourceBindings_help_add": "Legg til hjelpetekst",
"ux_editor.modal_properties_textResourceBindings_next": "Tekst på neste-knapp",
"ux_editor.modal_properties_textResourceBindings_next_add": "Legg til tekst på neste-knapp",
"ux_editor.modal_properties_textResourceBindings_save_and_next_button": "'Lagre og neste'-knapp",
"ux_editor.modal_properties_textResourceBindings_save_and_next_button_add": "Legg til tekst for 'Lagre og neste'-knapp",
"ux_editor.modal_properties_textResourceBindings_save_button": "'Lagre'-knapp",
"ux_editor.modal_properties_textResourceBindings_save_button_add": "'Legg til tekst for Lagre'-knapp",
"ux_editor.modal_properties_textResourceBindings_shortName": "Kortnavn",
"ux_editor.modal_properties_textResourceBindings_shortName_add": "Legg til kortnavn",
"ux_editor.modal_properties_textResourceBindings_tableTitle": "Tittel i tabell",
"ux_editor.modal_properties_textResourceBindings_tableTitle_add": "Legg til tittel i tabell",
"ux_editor.modal_properties_textResourceBindings_tag": "Merkebeskrivelse:",
"ux_editor.modal_properties_textResourceBindings_tag_add": "Legg til merkebeskrivelse",
"ux_editor.modal_properties_textResourceBindings_tagTitle": "Merkebeskrivelse",
"ux_editor.modal_properties_textResourceBindings_tagTitle_add": "Legg til merkebeskrivelse",
"ux_editor.modal_properties_textResourceBindings_title": "Ledetekst",
"ux_editor.modal_properties_textResourceBindings_title_add": "Legg til ledetekst",
"ux_editor.modal_properties_trigger_validation_label": "Skal feltet trigge en validering?",
Expand Down Expand Up @@ -1661,6 +1673,13 @@
"ux_editor.pages_error_length": "Navnet kan ikke være lengre enn 30 tegn.",
"ux_editor.pages_error_unique": "Navnet må være unikt.",
"ux_editor.preview": "Forhåndsvisning",
"ux_editor.properties_panel.options.add_options": "Legg til alternativer",
"ux_editor.properties_panel.options.codelist_switch_to_custom": "Bytt til egendefinert kodeliste",
"ux_editor.properties_panel.options.codelist_switch_to_static": "Bytt til statisk kodeliste",
"ux_editor.properties_panel.options.remove_option": "Slett alternativ",
"ux_editor.properties_panel.options.use_code_list_helptext": "Skru på denne innstillingen for å bruke en kodeliste for å styre alternativene til denne komponenten. Skru av innstillingen for å legge til alternativene manuelt.",
"ux_editor.properties_panel.options.use_code_list_label": "Bruk kodeliste",
"ux_editor.properties_panel.texts.options_title": "Kodelister og alternativer",
"ux_editor.radios_description_add": "Legg til beskrivelse",
"ux_editor.radios_description_placeholder": "Ingen beskrivelse",
"ux_editor.radios_error_DuplicateValues": "Alle verdier må være unike.",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -114,8 +114,8 @@ export const configComponents: IConfigComponents = {
component={component}
handleComponentChange={handleComponentChange}
textKey={EditSettings.TagTitle}
labelKey='ux_editor.modal_properties_textResourceBindings_tag'
placeholderKey='ux_editor.modal_properties_textResourceBindings_tag_add'
labelKey='ux_editor.modal_properties_textResourceBindings_tagTitle'
placeholderKey='ux_editor.modal_properties_textResourceBindings_tagTitle_add'
/>
),
[EditSettings.Options]: EditOptions,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,24 +19,9 @@ import { useLayoutSchemaQuery } from '../../hooks/queries/useLayoutSchemaQuery';

const user = userEvent.setup();

// Test data:
const textResourceEditTestId = 'text-resource-edit';

// Mocks:
jest.mock('../TextResourceEdit', () => ({
TextResourceEdit: () => <div data-testid={textResourceEditTestId} />,
}));

describe('ContentTab', () => {
afterEach(jest.clearAllMocks);

describe('when editing a text resource', () => {
it('should render the component', async () => {
await render({ props: {}, editId: 'test' });
expect(screen.getByTestId(textResourceEditTestId)).toBeInTheDocument();
});
});

describe('when editing a container', () => {
const props = {
formId: container1IdMock,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,20 +1,11 @@
import React from 'react';
import { TextResourceEdit } from '../TextResourceEdit';
import { EditFormComponent } from '../config/EditFormComponent';
import { EditFormContainer } from '../config/EditFormContainer';
import { getCurrentEditId } from '../../selectors/textResourceSelectors';
import { useSelector } from 'react-redux';
import { useFormContext } from '../../containers/FormContext';
import { useTranslation } from 'react-i18next';
import { isContainer } from '../../utils/formItemUtils';

export const Content = () => {
const { formId, form, handleUpdate, debounceSave } = useFormContext();
const editId = useSelector(getCurrentEditId);
const { t } = useTranslation();

if (editId) return <TextResourceEdit />;
if (!formId || !form) return t('right_menu.content_empty');

return isContainer(form) ? (
<EditFormContainer
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,21 +9,27 @@ import { formContextProviderMock } from '../../testing/formContextMocks';
const user = userEvent.setup();

// Test data:
const textText = 'Tekst';
const contentText = 'Innhold';
const dynamicsText = 'Dynamikk';
const calculationsText = 'Beregninger';
const texts = {
'right_menu.text': textText,
'right_menu.content': contentText,
'right_menu.dynamics': dynamicsText,
'right_menu.calculations': calculationsText,
};

const textTestId = 'text';
const contentTestId = 'content';
const conditionalRenderingTestId = 'conditional-rendering';
const expressionsTestId = 'expressions';
const calculationsTestId = 'calculations';

// Mocks:
jest.mock('./Text', () => ({
Text: () => <div data-testid={textTestId} />,
}));
jest.mock('./Content', () => ({
Content: () => <div data-testid={contentTestId} />,
}));
Expand All @@ -39,6 +45,23 @@ jest.mock('./Calculations', () => ({
jest.mock('react-i18next', () => ({ useTranslation: () => mockUseTranslation(texts) }));

describe('Properties', () => {
describe('Text', () => {
it('Toggles text when clicked', async () => {
render();
const button = screen.queryByRole('button', { name: textText });
await act(() => user.click(button));
expect(button).toHaveAttribute('aria-expanded', 'true');
await act(() => user.click(button));
expect(button).toHaveAttribute('aria-expanded', 'false');
});

it('Opens text when a component is selected', async () => {
const { rerender } = render();
rerender(getComponent({ formId: 'test' }));
const button = screen.queryByRole('button', { name: textText });
await waitFor(() => expect(button).toHaveAttribute('aria-expanded', 'true'));
});
});
describe('Content', () => {
it('Closes content on load', () => {
render();
Expand All @@ -54,13 +77,6 @@ describe('Properties', () => {
await act(() => user.click(button));
expect(button).toHaveAttribute('aria-expanded', 'false');
});

it('Opens content when a component is selected', async () => {
const { rerender } = render();
rerender(getComponent({ formId: 'test' }));
const button = screen.queryByRole('button', { name: contentText });
await waitFor(() => expect(button).toHaveAttribute('aria-expanded', 'true'));
});
});

describe('Dynamics', () => {
Expand Down Expand Up @@ -109,9 +125,11 @@ describe('Properties', () => {
it('Renders accordion', () => {
const formIdMock = 'test-id';
render({ formId: formIdMock });
expect(screen.getByText(textText)).toBeInTheDocument();
expect(screen.getByText(contentText)).toBeInTheDocument();
expect(screen.getByText(dynamicsText)).toBeInTheDocument();
expect(screen.getByText(calculationsText)).toBeInTheDocument();
expect(screen.getByTestId(textTestId)).toBeInTheDocument();
expect(screen.getByTestId(contentTestId)).toBeInTheDocument();
expect(screen.getByTestId(expressionsTestId)).toBeInTheDocument();
expect(screen.getByTestId(calculationsTestId)).toBeInTheDocument();
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
import React, { useEffect } from 'react';
import { Calculations } from './Calculations';
import { Content } from './Content';
import { Text } from './Text';
import { useTranslation } from 'react-i18next';
import { Accordion } from '@digdir/design-system-react';
import { useFormContext } from '../../containers/FormContext';
Expand All @@ -17,7 +18,7 @@ export const Properties = () => {
useEffect(() => {
if (formIdRef.current !== formId) {
formIdRef.current = formId;
if (formId && openList.length === 0) setOpenList(['content']);
if (formId && openList.length === 0) setOpenList(['text']);
}
}, [formId, openList.length]);

Expand All @@ -32,19 +33,27 @@ export const Properties = () => {
return (
<div className={classes.root}>
<Accordion color='subtle'>
<Accordion.Item open={openList.includes('text')}>
<Accordion.Header onHeaderClick={() => toggleOpen('text')}>
{t('right_menu.text')}
</Accordion.Header>
<Accordion.Content>{formId ? <Text /> : t('right_menu.content_empty')}</Accordion.Content>
</Accordion.Item>
<Accordion.Item open={openList.includes('content')}>
<Accordion.Header onHeaderClick={() => toggleOpen('content')}>
{t('right_menu.content')}
</Accordion.Header>
<Accordion.Content>
<Content />
{formId ? <Content /> : t('right_menu.content_empty')}
</Accordion.Content>
</Accordion.Item>
<Accordion.Item open={openList.includes('dynamics')}>
<Accordion.Header onHeaderClick={() => toggleOpen('dynamics')}>
{t('right_menu.dynamics')}
</Accordion.Header>
<Accordion.Content>{formId && <Dynamics />}</Accordion.Content>
<Accordion.Content>
{formId ? <Dynamics /> : t('right_menu.content_empty')}
</Accordion.Content>
</Accordion.Item>
<Accordion.Item open={openList.includes('calculations')}>
<Accordion.Header onHeaderClick={(e) => toggleOpen('calculations')}>
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
.textResourceContainer {
padding-bottom: 20px;
}
Loading