Skip to content

Commit 9ad2305

Browse files
authored
IBX-7124: Aligned confirmation/discard action labels in User settings with design (#1097)
1 parent fd6f3b4 commit 9ad2305

File tree

6 files changed

+24
-14
lines changed

6 files changed

+24
-14
lines changed

features/personas/ChangePassword.feature

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ Feature: Verify that an User allowed to change password can change his password
88
And I switch to "Account settings" tab in User settings
99
And I click on the change password button
1010
And I change password from "Passw0rd-42" to "Passw0rd-43"
11-
And I perform the "Update" action
11+
And I perform the "Save and close" action
1212
Then success notification that "Your password has been successfully changed." appears
1313
And I should be on "User settings" page
1414

features/standard/Autosave.feature

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ Feature: Content Items creation
3838
And I'm on Content view Page for root
3939
And I go to user settings
4040
And I disable autosave
41-
And I perform the "Save" action
41+
And I perform the "Save and close" action
4242
And I'm on Content view Page for root
4343
When I start creating a new content "Article"
4444
And I set content fields

src/bundle/Resources/translations/ibexa_menu.en.xliff

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -567,15 +567,25 @@
567567
<note>key: user_edit__sidebar_right__update</note>
568568
</trans-unit>
569569
<trans-unit id="d79a55ddd58ecf56b4edb39c794c858d649481e2" resname="user_password_change__sidebar_right__cancel">
570-
<source>Discard changes</source>
571-
<target state="new">Discard changes</target>
570+
<source>Discard</source>
571+
<target state="new">Discard</target>
572572
<note>key: user_password_change__sidebar_right__cancel</note>
573573
</trans-unit>
574574
<trans-unit id="ab8629add84cbd9cebfdf326f7242044ae7650af" resname="user_password_change__sidebar_right__update">
575-
<source>Update</source>
576-
<target state="new">Update</target>
575+
<source>Save and close</source>
576+
<target state="new">Save and close</target>
577577
<note>key: user_password_change__sidebar_right__update</note>
578578
</trans-unit>
579+
<trans-unit id="bbcc94ae51bb5c4c0a3222b554e32140bc542266" resname="user_setting_edit__sidebar_right__cancel">
580+
<source>Discard</source>
581+
<target state="new">Discard</target>
582+
<note>key: user_setting_edit__sidebar_right__cancel</note>
583+
</trans-unit>
584+
<trans-unit id="89bff903c56f538cb7ee5da8ac57a8eb8229707a" resname="user_setting_edit__sidebar_right__save">
585+
<source>Save and close</source>
586+
<target state="new">Save and close</target>
587+
<note>key: user_setting_edit__sidebar_right__save</note>
588+
</trans-unit>
579589
</body>
580590
</file>
581591
</xliff>

src/lib/EventListener/UserPasswordChangeRightSidebarListener.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,8 @@ public function onUserPasswordChangeRightSidebarConfigure(ConfigureMenuEvent $ev
7171
public static function getTranslationMessages(): array
7272
{
7373
return [
74-
(new Message(self::ITEM__UPDATE, 'ibexa_menu'))->setDesc('Update'),
75-
(new Message(self::ITEM__CANCEL, 'ibexa_menu'))->setDesc('Discard changes'),
74+
(new Message(self::ITEM__UPDATE, 'ibexa_menu'))->setDesc('Save and close'),
75+
(new Message(self::ITEM__CANCEL, 'ibexa_menu'))->setDesc('Discard'),
7676
];
7777
}
7878
}

src/lib/Menu/UserPasswordChangeRightSidebarBuilder.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -93,8 +93,8 @@ public function createStructure(array $options): ItemInterface
9393
public static function getTranslationMessages(): array
9494
{
9595
return [
96-
(new Message(self::ITEM__UPDATE, 'ibexa_menu'))->setDesc('Update'),
97-
(new Message(self::ITEM__CANCEL, 'ibexa_menu'))->setDesc('Discard changes'),
96+
(new Message(self::ITEM__UPDATE, 'ibexa_menu'))->setDesc('Save and close'),
97+
(new Message(self::ITEM__CANCEL, 'ibexa_menu'))->setDesc('Discard'),
9898
];
9999
}
100100
}

src/lib/Menu/UserSetting/UserSettingUpdateRightSidebarBuilder.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,8 @@
2525
class UserSettingUpdateRightSidebarBuilder extends AbstractBuilder implements TranslationContainerInterface
2626
{
2727
/* Menu items */
28-
public const ITEM__SAVE = 'object_state_edit__sidebar_right__save';
29-
public const ITEM__CANCEL = 'object_state_edit__sidebar_right__cancel';
28+
public const ITEM__SAVE = 'user_setting_edit__sidebar_right__save';
29+
public const ITEM__CANCEL = 'user_setting_edit__sidebar_right__cancel';
3030

3131
/** @var \Symfony\Contracts\Translation\TranslatorInterface */
3232
private $translator;
@@ -88,8 +88,8 @@ public function createStructure(array $options): ItemInterface
8888
public static function getTranslationMessages(): array
8989
{
9090
return [
91-
(new Message(self::ITEM__SAVE, 'ibexa_menu'))->setDesc('Save'),
92-
(new Message(self::ITEM__CANCEL, 'ibexa_menu'))->setDesc('Discard changes'),
91+
(new Message(self::ITEM__SAVE, 'ibexa_menu'))->setDesc('Save and close'),
92+
(new Message(self::ITEM__CANCEL, 'ibexa_menu'))->setDesc('Discard'),
9393
];
9494
}
9595
}

0 commit comments

Comments
 (0)