From fff1d833b932370e5fc97a5419468413199281c6 Mon Sep 17 00:00:00 2001
From: Oliver Eyton-Williams <ojeytonwilliams@gmail.com>
Date: Fri, 28 Jan 2022 09:02:07 +0100
Subject: [PATCH] fix: prevent scroll jumping on /settings (#44937)

---
 client/src/components/settings/Certification.js  |  4 +---
 client/src/components/settings/certification.css | 10 +++++-----
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/client/src/components/settings/Certification.js b/client/src/components/settings/Certification.js
index 44042c4bcd..91327f2e23 100644
--- a/client/src/components/settings/Certification.js
+++ b/client/src/components/settings/Certification.js
@@ -429,9 +429,7 @@ export class CertificationSettings extends Component {
     const { t } = this.props;
     return (
       <ScrollableAnchor id='certification-settings'>
-        {/* it's weird that we repeat the id, but the ScrollableAnchor does not
-        add any elements to the DOM and cannot be used for styling  */}
-        <section id='certification-settings'>
+        <section className='certification-settings'>
           <SectionHeader>{t('settings.headings.certs')}</SectionHeader>
           {certifications.map(certName =>
             this.renderCertifications(certName, projectMap)
diff --git a/client/src/components/settings/certification.css b/client/src/components/settings/certification.css
index 855a09b15f..421d2fc376 100644
--- a/client/src/components/settings/certification.css
+++ b/client/src/components/settings/certification.css
@@ -1,14 +1,14 @@
-#certification-settings .solutions-dropdown,
-#certification-settings .solutions-dropdown .dropdown-menu,
-#certification-settings .solutions-dropdown .dropdown {
+.certification-settings .solutions-dropdown,
+.certification-settings .solutions-dropdown .dropdown-menu,
+.certification-settings .solutions-dropdown .dropdown {
   width: 100%;
 }
 
-#certification-settings tr {
+.certification-settings tr {
   height: 57px;
 }
 
-#certification-settings .project-title > a {
+.certification-settings .project-title > a {
   line-height: 40px;
 }