Skip to content
This repository was archived by the owner on Sep 11, 2024. It is now read-only.

Commit afa8b01

Browse files
authored
Fix headings margin on security user settings tab (#8826)
* Apply kind=link to 'Learn more' link on security user settings tab Signed-off-by: Suguru Hirahara <[email protected]> * Remove specific margin setting from heading on security user settings tab Signed-off-by: Suguru Hirahara <[email protected]>
1 parent 2ebb5eb commit afa8b01

File tree

3 files changed

+6
-14
lines changed

3 files changed

+6
-14
lines changed

res/css/views/settings/tabs/_SettingsTab.scss

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -92,12 +92,6 @@ limitations under the License.
9292
}
9393
}
9494

95-
.mx_SettingsTab_linkBtn {
96-
cursor: pointer;
97-
color: $accent;
98-
word-break: break-all;
99-
}
100-
10195
.mx_SettingsTab_toggleWithDescription {
10296
margin-top: $spacing-24;
10397
}

res/css/views/settings/tabs/user/_SecurityUserSettingsTab.scss

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,6 @@ limitations under the License.
2727
}
2828

2929
.mx_SecurityUserSettingsTab {
30-
.mx_SettingsTab_heading {
31-
margin-bottom: 22px;
32-
}
3330
.mx_SettingsTab_section {
3431
.mx_AccessibleButton_kind_link {
3532
font-size: inherit;

src/components/views/settings/tabs/user/SecurityUserSettingsTab.tsx

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -306,11 +306,12 @@ export default class SecurityUserSettingsTab extends React.Component<IProps, ISt
306306
{ _t("Share anonymous data to help us identify issues. Nothing personal. " +
307307
"No third parties.") }
308308
</p>
309-
<p>
310-
<AccessibleButton className="mx_SettingsTab_linkBtn" onClick={onClickAnalyticsLearnMore}>
311-
{ _t("Learn more") }
312-
</AccessibleButton>
313-
</p>
309+
<AccessibleButton
310+
kind="link"
311+
onClick={onClickAnalyticsLearnMore}
312+
>
313+
{ _t("Learn more") }
314+
</AccessibleButton>
314315
</div>
315316
{ PosthogAnalytics.instance.isEnabled() && (
316317
<SettingsFlag

0 commit comments

Comments
 (0)