Phase 6#2
Conversation
MohammadAminDadkhah
commented
Jul 30, 2025
MohammadAminDadkhah
commented
…ove book detail responsiveness
…n panel and book detail components
…k component and books service
…tting across components
…te book components
| @@ -0,0 +1,3 @@ | |||
| main { | |||
| min-height: calc(100vh - 14rem); | |||
| path: '', | ||
| component: MainComponent, | ||
| children: [ | ||
| { path: 'home', redirectTo: '', pathMatch: 'full' }, |
There was a problem hiding this comment.
به نظرتون این اضافه کاری نیست که home رو بیایین ریدایرکت کنین به path خالی؟ اگه میخوایین وقتی مسیر اشتباهی وارد شد ریدایرکت بشه به home میتونین از wildcard استفاده کنین که اگه مسیر وارد شده با مسیر های تعریف شده شما match نشد به جایی که میخوایین کاربر رو هدایت کنید
There was a problem hiding this comment.
نه این رو معمولا میزارن که اگه طرف یا لینکی به /home اومد اتفاقا به not found page نره و همون / که همون home page هست رو نشونش بده
| providedIn: 'root', | ||
| }) | ||
| export class BooksService { | ||
| private books = BOOKS; |
There was a problem hiding this comment.
این کار رو به خاطر این انجام ندادیم چون initial value داشت ولی اگر مقدار اولیش خالی بود درست بود
| import { BooksService } from '../../../services/books.service'; | ||
|
|
||
| @Component({ | ||
| selector: 'app-top-bar', |
There was a problem hiding this comment.
بهتر بود اسم این کامبوننت رو top-bar نذارید. بر اساس کاری که داره انجام میده اسمش filter داشته باشه بهتره
There was a problem hiding this comment.
اره ولی این پیش بینی شده شاید button های دیگه هم اضافه بشه برا همین کلی در نظر گرفتیم بر اساس موقعیت جای گیریش
| this.books = this.booksService.getAllBooks; | ||
| } | ||
|
|
||
| ngDoCheck() { |
There was a problem hiding this comment.
به جای استفاده از ngDoCheck که بر اساس خیلی از رویداد هایی که توی صفحه و برنامه رخ میدن و به به سرچ مرتبط نیستن میتونستید یه event با هر تغییر input بدید و از اون استفاده کنید تا performance بهتری داشته باشید.
|
Screencast from 25-07-31 21:22:44.webm |
|
در کل یه سری جاها کد های تکراری ای زدید که میتونن در قالب یه کابوننت در بیان و اینکه کابوننت هایی که تعریف شدن هم میشه reusable تر بیاده سازی بشن |
refactor: simplify onCloseModal method in AddBookComponent
| display: flex; | ||
| align-items: center; | ||
| justify-content: center; | ||
| box-shadow: 0 -0.2rem 0.8rem rgba(255, 255, 255, 0.35); |
There was a problem hiding this comment.
توی box-shadow واقعا نیازه که از rem استفاده بشه؟
| export const HEADER_LINK = [ | ||
| { | ||
| title: 'Home', | ||
| src: '', | ||
| }, | ||
| { | ||
| title: 'Add book', | ||
| src: '/admin', | ||
| }, | ||
| ]; |
There was a problem hiding this comment.
اسمش نباید با حرف بزرگ نوشته بشه و اینکه بجای _ باید از . استفاده بشه
header.links.ts
4 participants
