remove whitespaces and configuration settings refactor

SenseNet · Jul 9, 2024 · a95bb29 · a95bb29
1 parent 0db3462
commit a95bb29
Show file tree

Hide file tree

Showing 18 changed files with 245 additions and 87 deletions.
diff --git a/apps/sensenet/src/components/AddButton.tsx b/apps/sensenet/src/components/AddButton.tsx
@@ -47,7 +47,8 @@ const useStyles = makeStyles((theme: Theme) => {
       display: 'flex',
       alignItems: 'center',
       justifyContent: 'space-evenly',
-      height: globals.common.drawerItemHeight,
+      height: globals.common.addButtonHeight,
+      paddingLeft: '2px',
     },
     listDropdown: {
       padding: '10px 0 10px 10px',

diff --git a/apps/sensenet/src/components/PageTitle.tsx b/apps/sensenet/src/components/PageTitle.tsx
@@ -10,7 +10,8 @@ const useStyles = makeStyles((theme: Theme) => {
     pageTitle: {
       color: theme.palette.type === 'light' ? theme.palette.common.black : theme.palette.common.white,
       fontFamily: '"Larsseit", Roboto',
-      fontSize: '20px',
+      fontSize: '24px',
+      padding: '8px 16px',
     },
   })
 })

diff --git a/apps/sensenet/src/components/appbar/desktop-app-bar.tsx b/apps/sensenet/src/components/appbar/desktop-app-bar.tsx
@@ -21,7 +21,8 @@ const useStyles = makeStyles(() => {
     toolBar: {
       position: 'static',
       height: '100%',
-      paddingLeft: '32px',
+      minHeight: '42px',
+      paddingLeft: '16px',
       paddingRight: 0,
     },
     logo: {

diff --git a/apps/sensenet/src/components/appbar/desktop-nav-menu.tsx b/apps/sensenet/src/components/appbar/desktop-nav-menu.tsx
@@ -21,6 +21,7 @@ const useStyles = makeStyles((theme: Theme) =>
     viewOptions: {
       cursor: 'pointer',
       marginRight: '16px',
+      padding: '7px',
     },
     navMenu: {
       height: '100%',

diff --git a/apps/sensenet/src/components/command-palette/CommandPalette.tsx b/apps/sensenet/src/components/command-palette/CommandPalette.tsx
@@ -45,9 +45,8 @@ const useStyles = makeStyles(() => {
     },
     iconButton: {
       color: globals.common.headerText,
-      '&:hover': {
-        backgroundColor: 'initial',
-      },
+      padding: '7px',
+      marginRight: '6px',
     },
     comboBox: {
       position: 'relative',

diff --git a/apps/sensenet/src/components/content-list/content-list.tsx b/apps/sensenet/src/components/content-list/content-list.tsx
@@ -93,7 +93,7 @@ export const isReferenceField = (fieldName: string, repo: Repository, schema = '
 }
 
 const rowHeightConst = 67
-const headerHeightConst = 58
+const headerHeightConst = 48
 
 /**
  * Compare passed minutes with
@@ -747,17 +747,16 @@ export const ContentList = <T extends GenericContent = GenericContent>(props: Co
               rowStyle: {
                 position: 'relative',
                 top: 'unset',
-                height: 'auto',
+                height: '48px',
                 overflow: 'initial',
-                padding: '5px 0px',
+                padding: '0',
               },
               onRowDoubleClick: onItemDoubleClickFunc,
               disableHeader: props.hideHeader,
               containerStyle: {
                 display: 'flex',
                 flexDirection: 'column',
                 overflowY: 'auto',
-                paddingBottom: '15px',
                 minHeight: '100%',
                 height: 'inherit',
                 maxHeight: 'inherit',

diff --git a/apps/sensenet/src/components/content/Explore.tsx b/apps/sensenet/src/components/content/Explore.tsx
@@ -50,7 +50,6 @@ const useStyles = makeStyles((theme: Theme) => {
       width: '100%',
       position: 'relative',
       overflow: 'hidden',
-      paddingTop: '8px',
     },
   })
 })

diff --git a/apps/sensenet/src/components/context-menu/content-context-menu.tsx b/apps/sensenet/src/components/context-menu/content-context-menu.tsx
@@ -79,7 +79,7 @@ export const ContentContextMenu: React.FunctionComponent<ContentContextMenuProps
           disablePortal={true}
           open={props.isOpened}
           PaperProps={{ style: { paddingBottom: '2em' } }}>
-          <List>
+          <List style={{ padding: 0, backgroundColor: 'green' }}>
             {actions?.map((action) => {
               return (
                 <ListItem

diff --git a/apps/sensenet/src/components/drawer/PermanentDrawer.tsx b/apps/sensenet/src/components/drawer/PermanentDrawer.tsx
@@ -52,7 +52,8 @@ const useStyles = makeStyles((theme: Theme) => {
       width: '100%',
     },
     listButton: {
-      height: '65px',
+      height: '60px',
+      paddingLeft: '2px',
     },
     expandCollapseWrapper: {
       height: '49px',
@@ -126,7 +127,7 @@ export const PermanentDrawer = () => {
         </List>
 
         {systemItems && (
-          <List>
+          <List style={{ padding: 0 }}>
             {systemItems.map((item) => {
               return (
                 <li key={item.itemType}>

diff --git a/apps/sensenet/src/components/drawer/PermanentDrawerItem.tsx b/apps/sensenet/src/components/drawer/PermanentDrawerItem.tsx
@@ -30,7 +30,8 @@ const useStyles = makeStyles((theme: Theme) => {
       },
     },
     listButton: {
-      height: '65px',
+      height: '60px',
+      paddingLeft: '2px',
     },
     listItemIconDark: {
       color: theme.palette.common.white,

diff --git a/apps/sensenet/src/components/event-list/event-list.tsx b/apps/sensenet/src/components/event-list/event-list.tsx
@@ -54,7 +54,7 @@ export default function EventList() {
         {currentEvent ? (
           <EventDetails event={currentEvent} />
         ) : (
-          <List style={{ height: 'calc(100% - 48px)', overflow: 'auto' }} values={events} />
+          <List style={{ height: 'calc(100% - 48px)', overflow: 'auto', backgroundColor: 'green' }} values={events} />
         )}
       </div>
     </FilterContextProvider>

diff --git a/apps/sensenet/src/components/layout/DesktopLayout.tsx b/apps/sensenet/src/components/layout/DesktopLayout.tsx
@@ -35,10 +35,7 @@ const useStyles = makeStyles((theme: Theme) => {
       boxSizing: 'border-box',
       overflow: 'hidden',
       height: '100%',
-      paddingTop: '10px',
-      '& > *': {
-        paddingLeft: `${globals.common.explorePaddingLeft}`,
-      },
+      padding: '0px',
     },
     executeActionPaper: {
       height: '100%',

diff --git a/apps/sensenet/src/components/settings/content-card.tsx b/apps/sensenet/src/components/settings/content-card.tsx
@@ -1,4 +1,15 @@
-import { createStyles, IconButton, makeStyles, Theme, Tooltip } from '@material-ui/core'
+import {
+  createStyles,
+  IconButton,
+  makeStyles,
+  Table,
+  TableCell,
+  TableContainer,
+  TableHead,
+  TableRow,
+  Theme,
+  Tooltip,
+} from '@material-ui/core'
 import Button from '@material-ui/core/Button'
 import Card from '@material-ui/core/Card'
 import CardActions from '@material-ui/core/CardActions'
@@ -82,67 +93,70 @@ export const ContentCard = ({ settings, onContextMenu }: ContentCardProps) => {
   const dataTestName = settingsTitle.replace(/\s+/g, '-').toLowerCase()
 
   return (
-    <Card
-      onContextMenu={(ev) => {
-        ev.preventDefault()
-        onContextMenu(ev)
-      }}
-      className={classes.card}
-      data-test={`content-card-${settingsName.replace(/\s+/g, '-').toLowerCase()}`}>
-      <CardContent>
-        <Tooltip placement="top" title={settingsName}>
-          <Typography variant="h5" gutterBottom={true} className={classes.title}>
-            {settingsTitle}
-          </Typography>
-        </Tooltip>
-        <Typography
-          color="textSecondary"
-          style={{ wordWrap: 'break-word' }}
-          dangerouslySetInnerHTML={{ __html: settings.Description || '' }}
-        />
-      </CardContent>
-      <CardActions style={{ justifyContent: 'flex-end' }}>
-        {!isSystemSettings.includes(settingsTitle) && (
-          <IconButton
-            data-test={`${dataTestName}-delete-button`}
-            aria-label="delete"
-            onClick={() => {
-              openDialog({
-                name: 'delete',
-                props: { content: [settings] },
-                dialogProps: { disableBackdropClick: true, disableEscapeKeyDown: true },
-              })
-            }}>
-            <DeleteIcon />
-          </IconButton>
-        )}
-        <Link
-          to={getPrimaryActionUrl({ content: settings, repository, uiSettings, location: history.location })}
-          style={{ textDecoration: 'none' }}>
-          <Button
-            aria-label={localization.edit}
-            size="small"
-            className={classes.button}
-            data-test={`${dataTestName}-edit-button`}>
-            {localization.edit}
-          </Button>
-        </Link>
-        {hasDocumentation.includes(settingsTitle) && (
-          <a
-            target="_blank"
-            rel="noopener noreferrer"
-            href={`${SETUP_DOCS_URL}${createAnchorFromName(settings.Name)}`}
+    <div>
+      <div />
+      <Card
+        onContextMenu={(ev) => {
+          ev.preventDefault()
+          onContextMenu(ev)
+        }}
+        className={classes.card}
+        data-test={`content-card-${settingsName.replace(/\s+/g, '-').toLowerCase()}`}>
+        <CardContent>
+          <Tooltip placement="top" title={settingsName}>
+            <Typography variant="h5" gutterBottom={true} className={classes.title}>
+              {settingsTitle}
+            </Typography>
+          </Tooltip>
+          <Typography
+            color="textSecondary"
+            style={{ wordWrap: 'break-word' }}
+            dangerouslySetInnerHTML={{ __html: settings.Description || '' }}
+          />
+        </CardContent>
+        <CardActions style={{ justifyContent: 'flex-end' }}>
+          {!isSystemSettings.includes(settingsTitle) && (
+            <IconButton
+              data-test={`${dataTestName}-delete-button`}
+              aria-label="delete"
+              onClick={() => {
+                openDialog({
+                  name: 'delete',
+                  props: { content: [settings] },
+                  dialogProps: { disableBackdropClick: true, disableEscapeKeyDown: true },
+                })
+              }}>
+              <DeleteIcon />
+            </IconButton>
+          )}
+          <Link
+            to={getPrimaryActionUrl({ content: settings, repository, uiSettings, location: history.location })}
             style={{ textDecoration: 'none' }}>
             <Button
-              aria-label={localization.learnMore}
+              aria-label={localization.edit}
               size="small"
               className={classes.button}
-              data-test={`${dataTestName}-learnmore-button`}>
-              {localization.learnMore}
+              data-test={`${dataTestName}-edit-button`}>
+              {localization.edit}
             </Button>
-          </a>
-        )}
-      </CardActions>
-    </Card>
+          </Link>
+          {hasDocumentation.includes(settingsTitle) && (
+            <a
+              target="_blank"
+              rel="noopener noreferrer"
+              href={`${SETUP_DOCS_URL}${createAnchorFromName(settings.Name)}`}
+              style={{ textDecoration: 'none' }}>
+              <Button
+                aria-label={localization.learnMore}
+                size="small"
+                className={classes.button}
+                data-test={`${dataTestName}-learnmore-button`}>
+                {localization.learnMore}
+              </Button>
+            </a>
+          )}
+        </CardActions>
+      </Card>
+    </div>
   )
 }