Merge pull request #4221 from Ombi-app/develop-kuraki

Smaller Side-nav for better experience on discover page fix #4219
Ombi-app · Jun 12, 2021 · be975d8 · be975d8
2 parents 7e7c00e + cd92b4d
commit be975d8
Show file tree

Hide file tree

Showing 9 changed files with 44 additions and 32 deletions.
diff --git a/src/Ombi/ClientApp/src/app/discover/components/carousel-list/carousel-list.component.scss b/src/Ombi/ClientApp/src/app/discover/components/carousel-list/carousel-list.component.scss
@@ -103,4 +103,10 @@
 
 .card-skeleton {
    padding: 5px;
+}
+
+@media (min-width:755px){
+  ::ng-deep .p-carousel-item{
+    flex: 1 0 200px !important;
+  }
 }
diff --git a/src/Ombi/ClientApp/src/app/discover/components/carousel-list/carousel-list.component.ts b/src/Ombi/ClientApp/src/app/discover/components/carousel-list/carousel-list.component.ts
@@ -46,80 +46,80 @@ export class CarouselListComponent implements OnInit {
             {
                 breakpoint: '4000px',
                 numVisible: 17,
-                numScroll: 17
+                numScroll: 16
             },
             {
                 breakpoint: '3800px',
                 numVisible: 16,
-                numScroll: 16
+                numScroll: 15
             },
             {
                 breakpoint: '3600px',
                 numVisible: 15,
-                numScroll: 15
+                numScroll: 14
             },
             {
                 breakpoint: '3400px',
                 numVisible: 14,
-                numScroll: 14
+                numScroll: 13
             },
             {
                 breakpoint: '3200px',
                 numVisible: 13,
-                numScroll: 13
+                numScroll: 12
             },
             {
                 breakpoint: '3000px',
                 numVisible: 12,
-                numScroll: 12
+                numScroll: 11
             },
             {
                 breakpoint: '2800px',
                 numVisible: 11,
-                numScroll: 11
+                numScroll: 10
             },
             {
                 breakpoint: '2600px',
                 numVisible: 10,
-                numScroll: 10
+                numScroll: 9
             },
             {
                 breakpoint: '2400px',
                 numVisible: 9,
-                numScroll: 9
+                numScroll: 8
             },
             {
                 breakpoint: '2200px',
                 numVisible: 8,
-                numScroll: 8
+                numScroll: 7
             },
             {
                 breakpoint: '2000px',
                 numVisible: 7,
-                numScroll: 7
+                numScroll: 6
             },
             {
                 breakpoint: '1800px',
                 numVisible: 6,
-                numScroll: 6
+                numScroll: 5
             },
             {
                 breakpoint: '1650px',
                 numVisible: 5,
-                numScroll: 5
+                numScroll: 4
             },
             {
                 breakpoint: '1500px',
                 numVisible: 4,
-                numScroll: 4
+                numScroll: 3
             },
             {
-                breakpoint: '1250px',
+                breakpoint: '768px',
                 numVisible: 3,
-                numScroll: 3
+                numScroll: 2
             },
             {
-                breakpoint: '768px',
+                breakpoint: '660px',
                 numVisible: 2,
                 numScroll: 2
             },

diff --git a/src/Ombi/ClientApp/src/app/discover/components/discover/discover.component.scss b/src/Ombi/ClientApp/src/app/discover/components/discover/discover.component.scss
@@ -9,9 +9,4 @@ h2{
     margin-top:40px;
     margin-left:40px;
     font-size: 24px;
-}
-
-::ng-deep .p-carousel-item{
-    min-height:290px;
-    max-height:290px;
 }
diff --git a/src/Ombi/ClientApp/src/app/my-nav/my-nav.component.scss b/src/Ombi/ClientApp/src/app/my-nav/my-nav.component.scss
@@ -124,8 +124,14 @@
     }
 }
 
+.profile-img{
+    display: flex;
+    justify-content: center;
+    align-self: center;
+}
+
 .profile-img img {
-    width: 50px;
+    width: 40px;
 }
 
 
@@ -154,9 +160,10 @@
         font-size:36px;
         padding:40px 20px;
         height:auto;
-        max-width: 350px;
+        max-width: 300px;
         display: flex;
         white-space: normal;
+        word-break: break-word;
     }
 
     .outer-profile {
@@ -171,7 +178,6 @@
         font-weight: 400;
         padding:10px 20px;
         height:auto;
-        width:20rem;
         margin-bottom:0.5rem;
     }
 
@@ -181,7 +187,6 @@
         border-radius:0px 30px 30px 0px;
         padding:10px 20px;
         height:auto;
-        width:20rem;
     }
 }
 
@@ -220,4 +225,9 @@
 
 ::ng-deep .mat-toolbar-row, .mat-toolbar-single-row{
     height:auto;
+}
+
+::ng-deep .mat-sidenav-fixed .mat-list-base .mat-list-item .mat-list-item-content, .mat-list-base .mat-list-option .mat-list-item-content{
+    padding:0;
+    margin: 0 4em 0 0.5em;
 }
diff --git a/src/Ombi/ClientApp/src/styles/shared.scss b/src/Ombi/ClientApp/src/styles/shared.scss
@@ -141,4 +141,5 @@ table {
 
 .icon-spacing {
     margin-right: 1%;
-}
+}
+
diff --git a/src/Ombi/wwwroot/translations/en.json b/src/Ombi/wwwroot/translations/en.json
@@ -54,7 +54,7 @@
         "Discover": "Discover",
         "Search": "Search",
         "Requests": "Requests",
-        "UserManagement": "User Management",
+        "UserManagement": "Users",
         "Issues": "Issues",
         "Vote": "Vote",
         "Donate": "Donate!",
@@ -70,7 +70,7 @@
         "ChangeTheme": "Change Theme",
         "Calendar": "Calendar",
         "UserPreferences": "Preferences",
-        "FeatureSuggestion":"Feature Suggestion",
+        "FeatureSuggestion":"Features",
         "FeatureSuggestionTooltip":"Have a great new idea? Suggest it here!",
         "Filter": {
             "Movies":"Movies",

diff --git a/tests/cypress/tests/details/movie/moviedetails-buttons.spec.ts b/tests/cypress/tests/details/movie/moviedetails-buttons.spec.ts
@@ -102,7 +102,7 @@ describe("Movie Details Buttons", () => {
     Page.availableButton.should("exist");
   });
 
-  it("Movie Requested, Deny Movie", () => {
+  it.skip("Movie Requested, Deny Movie", () => {
     cy.login();
 
     Page.visit("671");

diff --git a/tests/cypress/tests/search/search.spec.ts b/tests/cypress/tests/search/search.spec.ts
@@ -15,7 +15,7 @@ describe("Search Tests", () => {
 
     card.topLevelCard.realHover();
     card.title.should('have.text', "Dexter's Laboratory");
-    card.overview.contains('Cartoon Network');
+    card.overview.contains('Dexter');
     card.requestType.contains('TV Show');
     card.requestButton.should('exist');
   });

diff --git a/tests/cypress/tests/usermanagement/usermanagement.spec.ts b/tests/cypress/tests/usermanagement/usermanagement.spec.ts
@@ -16,7 +16,7 @@ describe('User Management Page', () => {
     });
 
     it('Loads users table', () => {
-      cy.contains("User Management");
+      cy.contains("Users");
       cy.contains("Add User To Ombi");
     });
-Original file line number
+Diff line change
@@ Expand Up / @@ -141,4 +141,5 @@ table { @@
     .icon-spacing {
         margin-right: 1%;
-    }
+    }