Fix/clean code (#347)

* fix(packages): removed unused packages

* fix(protected): removed protected in all files

* fix(styles): removed some mixins and scss files

* fix(classes): removed old classes

---------

Co-authored-by: Nazar Semets <nazar690@gmail.com>

Author: nsemets

package-lock.json

@@ -38,7 +38,6 @@
     "@angular/platform-browser": "^19.2.0",
     "@angular/platform-browser-dynamic": "^19.2.0",
     "@angular/router": "^19.2.0",
-    "@angular/service-worker": "^19.2.0",
     "@fortawesome/fontawesome-free": "^6.7.2",
     "@ngx-translate/core": "^16.0.4",
     "@ngx-translate/http-loader": "^16.0.1",

package.json

@@ -17,7 +17,7 @@ export class BreadcrumbComponent {
   private readonly router = inject(Router);
   private readonly route = inject(ActivatedRoute);
 
-  protected readonly url = toSignal(
+  readonly url = toSignal(
     this.router.events.pipe(
       filter((event) => event instanceof NavigationEnd),
       map(() => this.router.url),
@@ -26,7 +26,7 @@ export class BreadcrumbComponent {
     { initialValue: this.router.url }
   );
 
-  protected readonly routeData = toSignal(
+  readonly routeData = toSignal(
     this.router.events.pipe(
       filter((event) => event instanceof NavigationEnd),
       map(() => this.getCurrentRouteData()),
@@ -35,9 +35,9 @@ export class BreadcrumbComponent {
     { initialValue: { skipBreadcrumbs: false } as RouteData }
   );
 
-  protected readonly showBreadcrumb = computed(() => this.routeData()?.skipBreadcrumbs !== true);
+  readonly showBreadcrumb = computed(() => this.routeData()?.skipBreadcrumbs !== true);
 
-  protected readonly parsedUrl = computed(() =>
+  readonly parsedUrl = computed(() =>
     this.url()
       .split('?')[0]
       .split('/')

src/app/core/components/breadcrumb/breadcrumb.component.ts

@@ -6,7 +6,7 @@
       </div>
     </div>
 
-    <div class="footer-socials flex justify-content-center align-items-center" [class]="isWeb() ? 'm-y-12' : 'm-y-24'">
+    <div class="footer-socials flex justify-content-center align-items-center" [class]="isWeb() ? 'my-3' : 'my-4'">
       @for (icon of socialIcons; track icon.name) {
         <a
           class="social-link flex align-items-center justify-content-center"
@@ -21,9 +21,9 @@
 
   <div
     class="footer-secondary-nav flex flex-column justify-content-center align-items-center md:flex-row md:justify-content-between"
-    [class.m-t-24]="!isWeb()"
+    [class.mt-4]="!isWeb()"
   >
-    <div class="footer-links flex flex-wrap justify-content-center m-t-4">
+    <div class="footer-links flex flex-wrap justify-content-center mt-1">
       <a routerLink="/terms-of-use">{{ 'footer.links.termsOfUse' | translate }}</a>
 
       <span class="separator">|</span>
@@ -43,6 +43,6 @@
       <a href="https://www.cos.io/initiatives/top-guidelines">{{ 'footer.links.topGuidelines' | translate }}</a>
     </div>
 
-    <div class="flex justify-content-center m-y-12">{{ 'footer.copyright' | translate }}</div>
+    <div class="flex justify-content-center my-3">{{ 'footer.copyright' | translate }}</div>
   </div>
 </footer>

src/app/core/components/footer/footer.component.html

@@ -18,5 +18,5 @@ import { IS_WEB } from '@osf/shared/helpers';
 export class FooterComponent {
   isWeb = toSignal(inject(IS_WEB));
 
-  protected readonly socialIcons = SOCIAL_ICONS;
+  readonly socialIcons = SOCIAL_ICONS;
 }

src/app/core/components/footer/footer.component.ts

@@ -38,7 +38,7 @@ export class NavMenuComponent {
   private readonly isAuthenticated = select(UserSelectors.isAuthenticated);
   private readonly currentResource = select(CurrentResourceSelectors.getCurrentResource);
 
-  protected readonly mainMenuItems = computed(() => {
+  readonly mainMenuItems = computed(() => {
     const isAuthenticated = this.isAuthenticated();
     const filtered = filterMenuItems(MENU_ITEMS, isAuthenticated);
 
@@ -63,7 +63,7 @@ export class NavMenuComponent {
     return items;
   });
 
-  protected readonly currentRoute = toSignal(
+  readonly currentRoute = toSignal(
     this.router.events.pipe(
       filter((event): event is NavigationEnd => event instanceof NavigationEnd),
       map(() => this.getRouteInfo())
@@ -73,11 +73,11 @@ export class NavMenuComponent {
     }
   );
 
-  protected readonly currentResourceId = computed(() => this.currentRoute().resourceId);
-  protected readonly currentProviderId = computed(() => this.currentRoute().providerId);
-  protected readonly isCollectionsRoute = computed(() => this.currentRoute().isCollectionsWithId);
-  protected readonly isPreprintRoute = computed(() => this.currentRoute().isPreprintRoute);
-  protected readonly canUserViewReviews = select(UserSelectors.getCanViewReviews);
+  readonly currentResourceId = computed(() => this.currentRoute().resourceId);
+  readonly currentProviderId = computed(() => this.currentRoute().providerId);
+  readonly isCollectionsRoute = computed(() => this.currentRoute().isCollectionsWithId);
+  readonly isPreprintRoute = computed(() => this.currentRoute().isPreprintRoute);
+  readonly canUserViewReviews = select(UserSelectors.getCanViewReviews);
 
   private getRouteInfo() {
     const urlSegments = this.router.url.split('/').filter((segment) => segment);
@@ -115,6 +115,6 @@ export class NavMenuComponent {
     }
   }
 
-  protected readonly hasVisibleChildren = (item: MenuItem): boolean =>
+  readonly hasVisibleChildren = (item: MenuItem): boolean =>
     Array.isArray(item.items) && item.items.some((child) => !!child.visible);
 }

src/app/core/components/nav-menu/nav-menu.component.ts

@@ -1,4 +1,4 @@
 <aside class="custom-scrollbar flex flex-column p-4 h-full">
   <img ngSrc="assets/images/logo.svg" alt="OSF Logo" height="36" width="106" />
-  <osf-nav-menu class="m-t-36" />
+  <osf-nav-menu class="mt-5" />
 </aside>

src/app/core/components/sidenav/sidenav.component.html

@@ -45,26 +45,26 @@ export class InstitutionsSummaryComponent implements OnInit {
   rightsSearch = select(InstitutionsAdminSelectors.getSearchResults);
   rightsLoading = select(InstitutionsAdminSelectors.getSearchResultsLoading);
 
-  protected departmentLabels: string[] = [];
-  protected departmentDataset: DatasetInput[] = [];
+  departmentLabels: string[] = [];
+  departmentDataset: DatasetInput[] = [];
 
-  protected projectsLabels: string[] = [];
-  protected projectDataset: DatasetInput[] = [];
+  projectsLabels: string[] = [];
+  projectDataset: DatasetInput[] = [];
 
-  protected registrationsLabels: string[] = [];
-  protected registrationsDataset: DatasetInput[] = [];
+  registrationsLabels: string[] = [];
+  registrationsDataset: DatasetInput[] = [];
 
-  protected osfProjectsLabels: string[] = [];
-  protected osfProjectsDataset: DatasetInput[] = [];
+  osfProjectsLabels: string[] = [];
+  osfProjectsDataset: DatasetInput[] = [];
 
-  protected storageLabels: string[] = [];
-  protected storageDataset: DatasetInput[] = [];
+  storageLabels: string[] = [];
+  storageDataset: DatasetInput[] = [];
 
-  protected licenceLabels: string[] = [];
-  protected licenceDataset: DatasetInput[] = [];
+  licenceLabels: string[] = [];
+  licenceDataset: DatasetInput[] = [];
 
-  protected addonLabels: string[] = [];
-  protected addonDataset: DatasetInput[] = [];
+  addonLabels: string[] = [];
+  addonDataset: DatasetInput[] = [];
 
   private readonly actions = createDispatchMap({
     fetchDepartments: FetchInstitutionDepartments,

src/app/features/admin-institutions/pages/institutions-summary/institutions-summary.component.ts

@@ -23,16 +23,16 @@ import { ToastService } from '@shared/services';
 })
 export class AddToCollectionConfirmationDialogComponent {
   private toastService = inject(ToastService);
-  protected dialogRef = inject(DynamicDialogRef);
-  protected config = inject(DynamicDialogConfig);
-  protected destroyRef = inject(DestroyRef);
-  protected isSubmitting = signal<boolean>(false);
-  protected actions = createDispatchMap({
+  dialogRef = inject(DynamicDialogRef);
+  config = inject(DynamicDialogConfig);
+  destroyRef = inject(DestroyRef);
+  isSubmitting = signal<boolean>(false);
+  actions = createDispatchMap({
     createCollectionSubmission: CreateCollectionSubmission,
     updateProjectPublicStatus: UpdateProjectPublicStatus,
   });
 
-  protected handleAddToCollectionConfirm(): void {
+  handleAddToCollectionConfirm(): void {
     const payload = this.config.data.payload;
     const project = this.config.data.project;
 

src/app/features/collections/components/add-to-collection/add-to-collection-confirmation-dialog/add-to-collection-confirmation-dialog.component.ts

@@ -67,27 +67,27 @@ export class AddToCollectionComponent implements CanDeactivateComponent {
   private readonly dialogService = inject(DialogService);
   private readonly translateService = inject(TranslateService);
 
-  protected readonly AddToCollectionSteps = AddToCollectionSteps;
-
-  protected collectionMetadataForm = new FormGroup({});
-  protected isProviderLoading = select(CollectionsSelectors.getCollectionProviderLoading);
-  protected collectionProvider = select(CollectionsSelectors.getCollectionProvider);
-  protected selectedProject = select(ProjectsSelectors.getSelectedProject);
-  protected currentUser = select(UserSelectors.getCurrentUser);
-  protected providerId = signal<string>('');
-  protected allowNavigation = signal<boolean>(false);
-  protected projectMetadataSaved = signal<boolean>(false);
-  protected projectContributorsSaved = signal<boolean>(false);
-  protected collectionMetadataSaved = signal<boolean>(false);
-  protected stepperActiveValue = signal<number>(AddToCollectionSteps.SelectProject);
-  protected primaryCollectionId = computed(() => this.collectionProvider()?.primaryCollection?.id);
-  protected isProjectMetadataDisabled = computed(() => !this.selectedProject());
-  protected isProjectContributorsDisabled = computed(() => !this.selectedProject() || !this.projectMetadataSaved());
-  protected isCollectionMetadataDisabled = computed(
+  readonly AddToCollectionSteps = AddToCollectionSteps;
+
+  collectionMetadataForm = new FormGroup({});
+  isProviderLoading = select(CollectionsSelectors.getCollectionProviderLoading);
+  collectionProvider = select(CollectionsSelectors.getCollectionProvider);
+  selectedProject = select(ProjectsSelectors.getSelectedProject);
+  currentUser = select(UserSelectors.getCurrentUser);
+  providerId = signal<string>('');
+  allowNavigation = signal<boolean>(false);
+  projectMetadataSaved = signal<boolean>(false);
+  projectContributorsSaved = signal<boolean>(false);
+  collectionMetadataSaved = signal<boolean>(false);
+  stepperActiveValue = signal<number>(AddToCollectionSteps.SelectProject);
+  primaryCollectionId = computed(() => this.collectionProvider()?.primaryCollection?.id);
+  isProjectMetadataDisabled = computed(() => !this.selectedProject());
+  isProjectContributorsDisabled = computed(() => !this.selectedProject() || !this.projectMetadataSaved());
+  isCollectionMetadataDisabled = computed(
     () => !this.selectedProject() || !this.projectMetadataSaved() || !this.projectContributorsSaved()
   );
 
-  protected actions = createDispatchMap({
+  actions = createDispatchMap({
     getCollectionProvider: GetCollectionProvider,
     clearAddToCollectionState: ClearAddToCollectionState,
     createCollectionSubmission: CreateCollectionSubmission,