Brojd/polish insights (#3625)

# What this PR does
- remove duplicated inner page padding
- reorder items in sidebar

## Checklist

- [x] Unit, integration, and e2e (if applicable) tests updated
- [x] Documentation added (or `pr:no public docs` PR label added if not
required)
- [x] `CHANGELOG.md` updated (or `pr:no changelog` PR label added if not
required)

grafana-plugin/src/pages/insights/Insights.module.scss

@@ -0,0 +1,6 @@
+// Required to remove inner page padding since grafana-scenes doesn't support its style modification
+.insights div[class*='page-inner'] {
+  padding-left: 0;
+  padding-right: 0;
+  border: none;
+}

grafana-plugin/src/pages/insights/Insights.tsx

@@ -21,6 +21,7 @@ import Text from 'components/Text/Text';
 import { useStore } from 'state/useStore';
 import { DOCS_ROOT } from 'utils/consts';
 
+import styles from './Insights.module.scss';
 import { InsightsConfig } from './Insights.types';
 import getAlertGroupsByIntegrationScene from './scenes/AlertGroupsByIntegration';
 import getAlertGroupsByTeamScene from './scenes/AlertGroupsByTeam';
@@ -45,10 +46,10 @@ const Insights = observer(() => {
   const appScene = useSceneApp(() => getAppScene({ isOpenSource, datasource }));
 
   return (
-    <>
+    <div className={styles.insights}>
       <InsightsInfoAlert />
       <appScene.Component model={appScene} />
-    </>
+    </div>
   );
 });
 

grafana-plugin/src/plugin.json

@@ -88,16 +88,16 @@
     },
     {
       "type": "page",
-      "name": "Settings",
-      "path": "/a/grafana-oncall-app/settings",
+      "name": "Insights",
+      "path": "/a/grafana-oncall-app/insights",
       "role": "Viewer",
       "action": "grafana-oncall-app.other-settings:read",
       "addToNav": true
     },
     {
       "type": "page",
-      "name": "Insights",
-      "path": "/a/grafana-oncall-app/insights",
+      "name": "Settings",
+      "path": "/a/grafana-oncall-app/settings",
       "role": "Viewer",
       "action": "grafana-oncall-app.other-settings:read",
       "addToNav": true