[ACS-5540] Implemented the review comments

projects/aca-content/src/lib/components/details/details.component.scss

@@ -4,7 +4,7 @@ app-details-manager {
 
     .aca-close-details-button {
       margin-right: 15px;
-      margin-top: 2px;
+      margin-top: 12px;
       outline: none;
       border-radius: 4px;
 

projects/aca-shared/rules/src/navigation.rules.spec.ts

@@ -226,26 +226,6 @@ describe('navigation.evaluators', () => {
   });
 
   describe('isDetails', () => {
-    it('should return true if url ends with `/details`', () => {
-      const context: any = {
-        navigation: {
-          url: '/path/details'
-        }
-      };
-
-      expect(app.isDetails(context)).toBe(true);
-    });
-
-    it('should return true if url starts with `/details`', () => {
-      const context: any = {
-        navigation: {
-          url: '/details/path'
-        }
-      };
-
-      expect(app.isDetails(context)).toBe(true);
-    });
-
     it('should return true if url includes with `/details`', () => {
       const context: any = {
         navigation: {
@@ -255,6 +235,16 @@ describe('navigation.evaluators', () => {
 
       expect(app.isDetails(context)).toBe(true);
     });
+
+    it('should return false if url not includes with `/details`', () => {
+      const context: any = {
+        navigation: {
+          url: '/path'
+        }
+      };
+
+      expect(app.isDetails(context)).toBe(false);
+    });
   });
 
   describe('isRecentFiles', () => {

projects/aca-shared/rules/src/navigation.rules.ts

@@ -112,7 +112,7 @@ export function isLibraryContent(context: RuleContext): boolean {
 
 export function isDetails(context: RuleContext): boolean {
   const { url } = context.navigation;
-  return url && (url.endsWith('/details') || url.includes('/details/') || url.startsWith('/details'));
+  return url?.includes('/details');
 }
 
 /**