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.

Sign up for GitHub

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

Jump to bottom

Задачи по 4му занятию #4

Merged

jsru-1 merged 21 commits into js-tasks-ru:master from YuGulin:master

Jan 15, 2025

YuGulin commented Dec 26, 2024

[MeetupView]
[UiClock]
[Удаляемые Email-ы]
[Компонент-счётчик]
[Компоненты: погода в Средиземье]

Vombat added 10 commits

December 25, 2024 17:17


          4е занятие - 1я задача

6c219bb


          Merge branch 'master' of https://github.com/js-tasks-ru/vue-20241213_…

acddaea

…yuvgulin


          4е занятие - 2я задача

cb98718


          Merge branch 'master' of https://github.com/js-tasks-ru/vue-20241213_…

770e937

…yuvgulin


          4е занятие - 3я задача

2ef4a68


          Merge branch 'master' of https://github.com/js-tasks-ru/vue-20241213_…

6ca61dd

…yuvgulin


          4е занятие - 4я задача

a164aa6


          Merge branch 'master' of https://github.com/js-tasks-ru/vue-20241213_…

5f587b8

…yuvgulin


          4е занятие - 5я задача

b4e47a4


          4е занятие - 5я задача - поправил имена

2a11b27

Contributor

jsru-1 commented Dec 26, 2024

Добавляю преподавателя (@ShGKme) для код-ревью.

jsru-1 requested a review from ShGKme

December 26, 2024 23:04

ShGKme requested changes

View reviewed changes

03-components/30-removable-emails/MarkedEmailsApp.js Outdated Show resolved Hide resolved

03-components/50-weather-components/components/WeatherCard.js Outdated Show resolved Hide resolved

03-components/10-MeetupView/MeetupView.js Outdated Show resolved Hide resolved


          4е занятие - правки по ревью

22f8d7e

Contributor

jsru-1 commented Jan 14, 2025

Добавляю преподавателя (@ShGKme) для код-ревью.

jsru-1 requested a review from ShGKme

January 14, 2025 21:35


          4е занятие - правки по ревью 2

2f7ee6c

Contributor

jsru-1 commented Jan 14, 2025

Решение было обновлено, посмотрим что скажет @ShGKme


          4е занятие - правки по ревью 3

0c3e31e

Contributor

jsru-1 commented Jan 14, 2025

Решение было обновлено, посмотрим что скажет @ShGKme


          Merge branch 'master' of https://github.com/js-tasks-ru/vue-20241213_…

8962cab

…yuvgulin

Contributor

jsru-1 commented Jan 14, 2025

Проверьте, пожалуйста, ваше решение, не все тесты прошли (PR не будет принят до тех пор, пока все добавленные задачи не будут решены).


          5е занятие - задача 1 SFC

5ce3e4c

Contributor

jsru-1 commented Jan 15, 2025

Решение было обновлено, посмотрим что скажет @ShGKme


          Merge branch 'master' of https://github.com/js-tasks-ru/vue-20241213_…

31a4ae5

…yuvgulin

Contributor

jsru-1 commented Jan 15, 2025

Проверьте, пожалуйста, ваше решение, не все тесты прошли (PR не будет принят до тех пор, пока все добавленные задачи не будут решены).


          5е занятие - задача 2 MeetupCover с v-bind в CSS

45ed243

Contributor

jsru-1 commented Jan 15, 2025

Решение было обновлено, посмотрим что скажет @ShGKme

Vombat added 2 commits

January 15, 2025 19:15


          Merge branch 'master' of https://github.com/js-tasks-ru/vue-20241213_…

5899a8b

…yuvgulin


          4е занятие SFC - задача 3UiStretch

beee19a

Contributor

jsru-1 commented Jan 15, 2025

Решение было обновлено, посмотрим что скажет @ShGKme

Vombat added 2 commits

January 15, 2025 20:26


          Merge branch 'master' of https://github.com/js-tasks-ru/vue-20241213_…

9808c51

…yuvgulin


          4е занятие SFC - правки

f147d14

Contributor

jsru-1 commented Jan 15, 2025

Проверьте, пожалуйста, ваше решение, не все тесты прошли (PR не будет принят до тех пор, пока все добавленные задачи не будут решены).

ShGKme previously approved these changes

View reviewed changes

Contributor

ShGKme left a comment

Принято

03-components/10-MeetupView/MeetupView.js

+                                      <!-- Программа митапа -->
+                                      <!-- Или при пустой программе - сообщение "Программа пока пуста..." в UiAlert -->
+                                      <MeetupAgenda v-if="meetup?.agenda?.length" :agenda="meetup.agenda" />

Contributor

ShGKme Jan 15, 2025

meetup описан, как обязательный параметр, поэтому его не нужно проверять на null через ?.

04-sfc/20-MeetupCover-style-v-bind/MeetupCover.vue

+                  background-position: center;
+                  /* Если изображение присутствует - берём его из CSS переменной, установленной на элемент в шаблоне */
+                  /* Иначе выводим изображение по умолчанию - var(--default-cover) */
+                  background-image: v-bind('image ? "url(" + image + ")" : "var(--default-cover)"');

Contributor

ShGKme Jan 15, 2025

На мой взгляд, лучше такие вычисления перенести в computed, оставляя стили проще. В отличие от шаблона, обычно в стилях совсем нет вычислений и связи с JS, нет хорошей поддержки IDE. Удобнее иметь там только имя свойства, а само вычисление оставлять в компоненте. И не будет страшного сгенерированного при компиляции имени CSS переменной.

Contributor

ShGKme Jan 15, 2025

Первый фон затемнения с linear-gradient(0deg, rgba(0, 0, 0, 0.4), rgba(0, 0, 0, 0.4)) потерялся.

ShGKme dismissed their stale review

January 15, 2025 20:13

Не все задачи решены

jsru-1 merged commit a61614a into js-tasks-ru:master

1 check failed

Contributor

ShGKme commented Jan 15, 2025

Бот уже принял PR, но были решены не все задачи. Их можно будет закончить в следующем PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet