Skip to content

Базовый уровень#13

Open
Tatsiana-Drozd wants to merge 6 commits intoCodegirlSchool:mainfrom
Tatsiana-Drozd:hometask-7
Open

Базовый уровень#13
Tatsiana-Drozd wants to merge 6 commits intoCodegirlSchool:mainfrom
Tatsiana-Drozd:hometask-7

Conversation

@Tatsiana-Drozd
Copy link
Copy Markdown

No description provided.

script.js Outdated

myCard.querySelector('h1').textContent = title;
myCard.querySelector('p').textContent = description;
myCard.querySelector('.tags').textContent = tags;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Таня, tags - это массив, поэтому вставить через textContent . тут нужен метод перебора массива , для каждого тега создать спан, добавить ему класс , текст. Сделать выборку контейнера для тегов и вставить теги в этот контейнер

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

если встречается массив - тогда мы его перебираем с помощью метода forEach. Извлеченные элементы( dog и cat) сохраняем в переменную. А почему мы создаем именно тег спан?

script.js Outdated
const Card9 = makeCardByTemplate("Лежанка", "Идеальное место для отдыха!", ["cat", "dog"], 1500, "./img/9.jpeg");
const Card10 = makeCardByTemplate("Поилка для собак", "Возьмите с собой в путешествие", ["dog"], 800, "./img/10.jpeg");
const Card11 = makeCardByTemplate("Переноска", "Путешествуйте с комфортом!", ["cat", "dog"], 3500, "./img/11.jpeg");
const Card12 = makeCardByTemplate("Поводок для собак", "Для чудесных прогулок вместе", ["dog"], 800, "./img/12.jpeg");
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Таня, 1е - почему названия переменных с большой буквы?
А если товаров будет 1000? Представляешь какой будет код?)))
Есть массив items. Нужно сделать перебор массива, будет доступ к каждому элементы массива. Для каждого элемента создать карточку и вставить её на страницу.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

const sentence = str.trim().toLowerCase();
return sentence.includes();
}

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

я немного запуталась( может, я думаю не в том направлении(

  1. содержимое инпут сохранила в переменную
  2. далее мне нужно search "причесать"? // str.trim().toLowerCase()
  3. и потом уже в search.includes()? // а что в скобках includes вводить? Я могу только слово или фразу ввести. А я могу указать переменную в скобках? (item.title)
    Маша, я понимаю, что бред пишу) но мне так хочется разобраться

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants