hexlet-rus / runit

Online Editor Form Embedding
https://runit.hexlet.ru/
Other
34 stars 114 forks source link

[#496] Надпись "Бесплатный проект Хекслета" больше не меняет шрифт по… #499

Closed Maiiiiiiiiia closed 1 month ago

Maiiiiiiiiia commented 1 month ago

Снимок экрана 2024-06-07 в 11 54 57

fey commented 1 month ago

@Maiiiiiiiiia для чего закомментировали строчку? Если код не нужен, лучше его удалить. Восстановить можно из гита.

Вижу, что вы поменяли названия шрифвта, он используется в нескольких местах. Может его вынести в переменную? Может его вообще сделать дефолтным для всей страницы? (используются ли еще шрифты?)

Maiiiiiiiiia commented 1 month ago

строчку удалю!

используется три шрифта:

  1. "Nunito Sans Light"
  2. "Nunito Sans SemiBold"
  3. "Nunito Sans Bold"

Переменную могу сделать, но нельзя же удалить все шрифты и оставить один?

Maiiiiiiiiia commented 1 month ago

а но на 68 и 74 строчке не надо было менять шрифт, поторопилась, простите( 3 шрифта используется (один, но разной толщины)

fey commented 1 month ago

По сути ведь это один шрифт? Может быть его можно подключить как один и использовать нужное начертание там, где нужно?

Maiiiiiiiiia commented 1 month ago

а нужно было новую ветку создавать? скажите если что. (как можно понять это мои первый пулреквест, учусь!)

fey commented 1 month ago

да, конечно. Если с веткой удобно работать вплане, что изменения изолированы в ней. Если будете работать над другой задачей, то изменения не смешаются.

fey commented 1 month ago

Что-то конфликты пошли. Возможно ветка устарела сильно. вам нужно подтянуть изменения https://ru.hexlet.io/qna/git/questions/kak-aktualizirovat-vetku-v-pull-request

HelenOne commented 1 month ago

@Maiiiiiiiiia вроде всё отлично, но сможете еще задеплоить демку чтобы визуально оценить результат?

Maiiiiiiiiia commented 1 month ago

Скрины же достаточно?) Снимок экрана 2024-06-11 в 22 54 55 Снимок экрана 2024-06-11 в 22 54 29

fey commented 1 month ago

@HelenOne мержим?