TGViewer
Джун на фронте | IT Dev Log Джун на фронте | IT Dev Log @divatoz · 7.44K subscribers
Post #936 3.78K
Джун на фронте | IT Dev Log Пополнение портфолио Я так и не разобрался, почему таймер сбрасывается при паузе, но думаю этот баг не заметят. Чуть подправил псевдоадаптивную верстку и зафиксировал плеер снизу экрана, выглядит неплохо. Этакий минимализм. На все ушло 10 часов, так что…
Чума чуваки, тут code reviews подъехало

Плюсы:
— Хорошее описание в корне проекта;
— Проект развернут в мир. На этом все 🤣

Минусы:
— Не очень чисто в плане форматирования текста: где-то нет переноса строки, где-то между "if" и "(" есть пробел, где-то нет. Нужно подключить что-то вроде prettier;

— В store и компонентах используется JS, a в API TS;

— Общие стили такие как container лучше вынести в отдельный файл css и подключить к layout/app, или делать компонентами;

— tracksStore сделан через options api, хотя в компонентах используется composition api;

— Дважды вызывается fetchTracks, на onMounted плеера и в списке треков;

— Плеер должен быть изолирован от конкретного метода API, данные в него должны попасть либо по действию пользователя, либо при загрузке конкретной страницы;

— Трек в плеере подменяется при обновлении страницы, бросается в глаза если обновлять несколько раз подряд, либо убирать рандомность на клиенте, либо верстку с данными для плеера формировать на сервере;

— NuxtImg sizes лучше использовать либо с локальными изображениями, либо с кастомным провайдером. Сейчас в наборе srcset ссылка на одну и ту же картинку;

— Лишний click=selectTrack в TrackItem;

— formatDuration будет плохо показывать кол-во секунд если их меньше 10, нужно добивать 0 в начало;

— Для определения isPlaying нужно использовать ID трека, а не сравнивать указатели объектов (tracksStore.currentTrack === props.track);

— api/[path].ts условие path == "tracks" ? "tracks" : path всегда отдаст path.
  • 👍 16
  • 🔥 4
  • ❤ 2
More from @divatoz
  1. Oct 4, 2026Подключил антиспам на Jev, накидайте плз непристойностей в комментарии, помогите задебажит…
  2. Oct 2, 2026Карпаты рассказал, как теперь правильно общаться с LLM Совсем недавно переживали, что Андр…
  3. Sep 30, 2026Рабочий дневник: День 568 Как я mcdis.ru верстал 🚬 Сегодня расскажу про процессы, а не пр…
  4. Sep 29, 2026ну в целом, пока что удивительно блеклая презентация. клон Muse/Grok-bot, подписка за $500…
  5. Sep 26, 2026ИИ официально стоит дороже чем марихуана и фентанил, а через два года станет дороже кокаин…
  6. Sep 25, 2026Как я этот пост на год подписки обменял С поста про расшифровку созвонов ко мне пришёл мар…
Threads Profile ViewerView any public Threads profile without an account.Open ThreadLook →Writing with AI? Make it sound human.Metric37 rewrites AI drafts so they read naturally. Free AI detector, 1,500 words free.Try Metric37 →