Commit 91fd050e authored by Monica Basta's avatar Monica Basta Committed by Commit Bot

[SyncSetup]: Update Sync radio button.

This CL removes the sub label for 'Sync everything' and 'Customize
sync'. It also removes the segmentation line between the 2 options.

Bug: 1043122
Change-Id: I07d11da1025cfe34e07feb2456211f0572770725
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095008Reviewed-by: default avatarEsmael Elmoslimany <aee@chromium.org>
Commit-Queue: Monica Basta <msalama@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748207}
parent c9e59fd5
......@@ -177,15 +177,9 @@
<message name="IDS_SETTINGS_NON_PERSONALIZED_SERVICES_SECTION_LABEL" desc="Title for a section that shows options for various Google services">
Other Google services
</message>
<message name="IDS_SETTINGS_SYNC_EVERYTHING_HINT" desc="Hint for the radio button which causes all properties to be synced.">
Sync all data types including ones that might be added in the future
</message>
<message name="IDS_SETTINGS_CUSTOMIZE_SYNC" desc="Label for the radio button to customize sync data types.">
Customize sync
</message>
<message name="IDS_SETTINGS_CUSTOMIZE_SYNC_HINT" desc="Hint or the radio button to customize sync data types.">
Sync only selected data types
</message>
<message name="IDS_SETTINGS_SYNC_DATA" desc="Title for sync data types list.">
Sync data
</message>
......
......@@ -24,15 +24,6 @@
.list-item > div {
flex: 1;
}
cr-radio-button[name='sync-everything'] > .secondary {
border-bottom: var(--cr-separator-line);
padding-bottom: var(--cr-section-vertical-padding);
}
cr-radio-button[name='customize-sync'] {
padding-top: var(--cr-section-vertical-padding);
}
</style>
<template is="dom-if" if="[[!syncSetupFriendlySettings_]]">
......@@ -55,11 +46,9 @@
on-selected-changed="onSyncDataRadioSelectionChanged_">
<cr-radio-button name="sync-everything">
$i18n{syncEverythingCheckboxLabel}
<div class="secondary">$i18n{syncEverythingHint}</div>
</cr-radio-button>
<cr-radio-button name="customize-sync">
$i18n{customizeSyncLabel}
<div class="secondary">$i18n{customizeSyncHint}</div>
</cr-radio-button>
</cr-radio-group>
</div>
......
......@@ -120,9 +120,7 @@ void AddSyncControlsStrings(content::WebUIDataSource* html_source) {
IDS_AUTOFILL_ENABLE_PAYMENTS_INTEGRATION_CHECKBOX_LABEL},
{"nonPersonalizedServicesSectionLabel",
IDS_SETTINGS_NON_PERSONALIZED_SERVICES_SECTION_LABEL},
{"syncEverythingHint", IDS_SETTINGS_SYNC_EVERYTHING_HINT},
{"customizeSyncLabel", IDS_SETTINGS_CUSTOMIZE_SYNC},
{"customizeSyncHint", IDS_SETTINGS_CUSTOMIZE_SYNC_HINT},
{"syncData", IDS_SETTINGS_SYNC_DATA},
};
AddLocalizedStringsBulk(html_source, kLocalizedStrings);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment