Remove superfluous user theme link properties (#2058)

By definition, the NavItem has an icon, which can/should not be rendered. Depending on the browser, only a small margin is visible. The icon is therefore removed. The same applies to the hover-title, which would only be displayed for collapsed menus, but does not apply to submenu entries.

gradle/changelog/theme_nav_item.yaml

@@ -0,0 +1,2 @@
+- type: fixed
+  description: Remove superfluous user theme link properties ([#2058](https://github.com/scm-manager/scm-manager/pull/2058))

scm-ui/ui-webapp/public/locales/de/commons.json

@@ -97,10 +97,7 @@
     "error": "Fehler",
     "error-message": "'me' ist nicht definiert",
     "theme": {
-      "nav": {
-        "label": "Design",
-        "title": "Design wählen"
-      },
+      "navLink": "Design",
       "subtitle": "Design wählen",
       "submit": "Anwenden",
       "light": {

scm-ui/ui-webapp/public/locales/en/commons.json

@@ -98,10 +98,7 @@
     "error": "Error",
     "error-message": "'me' is undefined",
     "theme": {
-      "nav": {
-        "label": "Theme",
-        "title": "Choose your Theme"
-      },
+      "navLink": "Theme",
       "subtitle": "Choose your Theme",
       "submit": "Activate",
       "light": {

scm-ui/ui-webapp/src/containers/Profile.tsx

@@ -119,12 +119,7 @@ const Profile: FC = () => {
                 label={t("profile.settingsNavLink")}
                 title={t("profile.settingsNavLink")}
               >
-                <NavLink
-                  to={`${url}/settings/theme`}
-                  icon="fas fa-palette"
-                  label={t("profile.theme.nav.label")}
-                  title={t("profile.theme.nav.title")}
-                />
+                <NavLink to={`${url}/settings/theme`} label={t("profile.theme.navLink")} />
                 {mayChangePassword && (
                   <NavLink to={`${url}/settings/password`} label={t("profile.changePasswordNavLink")} />
                 )}