-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Boards board #41
Boards board #41
Conversation
igorkedzierawski
commented
Feb 19, 2024
{{ isHidden(sliceWrapper) ? "chevron_right" : "expand_more" }} | ||
</mat-icon> | ||
<div class="flex items-center space-x-1"> | ||
<img class="w-5 h-5 rounded-full" src="{{ slice.metadata.avatarUrl }}" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
brak alt
attribute
} | ||
</div> | ||
</div> | ||
|
||
<mat-menu #groupingMenu="matMenu"> | ||
@for (taskGrouping of TASK_GROUPINGS; track taskGrouping) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
błąd z typowaniem
taskid: "t4", | ||
assigneeAvatar: "/assets/corn-icons/icon-72x72.png", | ||
assigneeName: "Bob Johnson" | ||
protected filterString = ""; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
brak typowania
assigneeAvatar: "/assets/corn-icons/icon-72x72.png", | ||
assigneeName: "Bob Johnson" | ||
protected filterString = ""; | ||
protected taskGrouping = TaskGrouping.BY_ASSIGNEE; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
brak typowania
enum TaskGrouping { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
czemu ten enum
jest wewnątrz klasy. Powinien być w @enums
<div class="flex items-center justify-between"> | ||
<span class="text-gray-300 text-sm">{{ task.taskid }}</span> | ||
<change-assignee-menu [associatedTask]="task"> | ||
<img |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
alt
imports: [ | ||
MatCardModule, | ||
ChangeAssigneeMenuComponent, | ||
], | ||
templateUrl: './taskcard.component.html', | ||
}) | ||
export class TaskcardComponent { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TaskCard
|
||
@ViewChild('droplistInstance') private droplistInstance!: CdkDropList; | ||
|
||
getDroplistInstance() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- DropList
- typ
-
- Możesz wykorzystać typescriptowego
get
'a zmienną nazwiesz z prefixem_
i będzie to sensowniejsze
- Możesz wykorzystać typescriptowego
export class TasklistComponent { | ||
|
||
@Input() tasks: Task[] = []; | ||
@Input() siblingDroplistInstances: any[] = []; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DropList,
Czemu any[]
return this.droplistInstance; | ||
} | ||
|
||
protected droppedHandler(event: CdkDragDrop<Task[]>) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
type
…, segmented compound words and changed default value of TaskGrouping
…s and services into separate files and placed them inside appropriate packets
import { Task } from '@core/interfaces/boards/board/task.interface'; | ||
|
||
@Injectable() | ||
export class ModelService { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
return types
@@ -1,17 +1,17 @@ | |||
import { Injectable } from '@angular/core'; | |||
import { SliceComponent } from './slice.component'; | |||
import { SliceComponent } from '../../../../../pages/boards/board/slice/slice.component'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
to możesz zastąpić za pomocą @pages
@@ -0,0 +1,112 @@ | |||
import { Injectable, Input } from '@angular/core'; | |||
import { SliceDescriptor } from '../../../../../pages/boards/board/slice/slice_descriptor'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
to też @pages
@@ -3,15 +3,21 @@ import { CommonModule } from '@angular/common'; | |||
import { MatInputModule } from '@angular/material/input'; | |||
import { MatButtonModule } from '@angular/material/button'; | |||
import { MatMenuModule } from '@angular/material/menu'; | |||
import { SlicesModelService, TaskChangedColumnEvent, TaskChangedGroupEvent, TaskGrouper } from './slice/slices_model.service'; | |||
import { SliceService } from './slice/slice.service'; | |||
import { SlicesModelService } from '../../../core/services/boards/board/slice/slices_model.service'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@core
import { SlicesModelService, TaskChangedColumnEvent, TaskChangedGroupEvent, TaskGrouper } from './slice/slices_model.service'; | ||
import { SliceService } from './slice/slice.service'; | ||
import { SlicesModelService } from '../../../core/services/boards/board/slice/slices_model.service'; | ||
import { SliceService } from '../../../core/services/boards/board/slice/slice.service'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@core
import { CommonModule } from '@angular/common'; | ||
import { MatInputModule } from '@angular/material/input'; | ||
import { MatButtonModule } from '@angular/material/button'; | ||
import { MatDividerModule } from '@angular/material/divider'; | ||
import { ModelService } from '../model.service'; | ||
import { BoardModelService } from '../../../../core/services/boards/board/model.service'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -1,11 +1,11 @@ | |||
import { AfterViewInit, Component, Input, OnDestroy, ViewChild } from '@angular/core'; | |||
import { CommonModule } from '@angular/common'; | |||
import { TaskListComponent } from '../task_list/task_list.component'; | |||
import { SliceService } from './slice.service'; | |||
import { SliceService } from '../../../../core/services/boards/board/slice/slice.service'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.