Skip to content

Commit f37fe57

Browse files
committed
ready for meets?
1 parent 0f9cb89 commit f37fe57

File tree

2 files changed

+36
-52
lines changed

2 files changed

+36
-52
lines changed

README.md

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,10 @@
11
# sahvsergio-Treehouse_Full-Stack_JavaScript_Techdegree_unit5
2+
[]Functionality has been added to switch back and forth between employees when the detail modal window is open.
3+
No errors in the console when the end or beginning of the list is reached.
4+
[]At least one of the following has been added or changed:
5+
color
6+
background color
7+
font
8+
box or text shadows
9+
10+
Above style changes noted in the README.md file

js/scripts.js

Lines changed: 27 additions & 52 deletions
Original file line numberDiff line numberDiff line change
@@ -5,34 +5,24 @@ let peopleUrl = "https://randomuser.me/api/?results=12&nat=AU,CA,GB,IE,NZ,US";
55
let searchContainer = document.querySelector(".search-container");
66
let galleryDiv = document.querySelector("#gallery");
77

8-
9-
108
// =================
119

1210
//fetch requests
1311

1412
//================
1513
async function fetchEmployees(url) {
1614
try {
17-
let response = await fetch(url)
18-
let data= await response.json();
19-
let employees=data.results;
20-
displayEmployees(employees)
15+
let response = await fetch(url);
16+
let data = await response.json();
17+
let employees = data.results;
18+
displayEmployees(employees);
2119
createModal(employees);
22-
23-
24-
25-
} catch (error){
26-
console.error(error);
27-
}
28-
29-
finally {
20+
} catch (error) {
21+
console.error(error);
22+
} finally {
3023
createSearch();
3124
closeModal();
3225
searchEmployee();
33-
34-
35-
3626
}
3727
}
3828

@@ -50,7 +40,7 @@ function createSearch() {
5040
//=======================================================
5141

5242
function displayEmployees(employees) {
53-
console.log(employees);
43+
5444
for (i = 0; i < employees.length; i++) {
5545
let employee = employees[i];
5646
let employeeMarkup = `<div class="card">
@@ -72,18 +62,16 @@ function createModal(employees) {
7262
galleryDiv.addEventListener("click", (e) => {
7363
let employeeCard = e.target.closest(".card");
7464
if (!employeeCard) return; //
75-
65+
7666
let employeeCardName = employeeCard.querySelector("h3").innerText;
77-
78-
67+
7968
let emp;
80-
69+
8170
const matchedEmployee = employees.find(
8271
(emp) => `${emp.name.first} ${emp.name.last}` === employeeCardName
8372
);
84-
let dob=matchedEmployee.dob.date.split('T');
85-
let dateOfBirth=dob[0].split('-');
86-
73+
let dob = matchedEmployee.dob.date.split("T");
74+
let dateOfBirth = dob[0].split("-");
8775

8876
let ModalMarkup = `<div class="modal-container">
8977
<div class="modal">
@@ -101,18 +89,11 @@ function createModal(employees) {
10189
</div>`;
10290
galleryDiv.insertAdjacentHTML("afterend", ModalMarkup);
10391
});
104-
105-
10692
}
10793

108-
109-
110-
111-
11294
fetchEmployees(peopleUrl);
11395

114-
function closeModal(){
115-
96+
function closeModal() {
11697
document.addEventListener("click", (e) => {
11798
const modalContainer = document.querySelector(".modal-container");
11899

@@ -123,24 +104,18 @@ function closeModal(){
123104
});
124105
}
125106

126-
function searchEmployee(){
127-
let searchInput=document.querySelector('#search-input');
128-
searchInput.addEventListener('keyup',(e)=>{
129-
let currentValue=e.target.value.toLowerCase();
130-
console.log(currentValue);
107+
function searchEmployee() {
108+
let searchInput = document.querySelector("#search-input");
109+
searchInput.addEventListener("keyup", (e) => {
110+
let currentValue = e.target.value.toLowerCase();
111+
131112
let employees = document.querySelectorAll("h3.card-name");
132-
employees.forEach(employee=>{
133-
if(employee.textContent.toLowerCase().includes(currentValue)){
134-
employee.parentNode.parentNode.style.display='block';
135-
136-
}
137-
else{
138-
employee.parentNode.parentNode.style.display='none';
113+
employees.forEach((employee) => {
114+
if (employee.textContent.toLowerCase().includes(currentValue)) {
115+
employee.parentNode.parentNode.style.display = "block";
116+
} else {
117+
employee.parentNode.parentNode.style.display = "none";
139118
}
140-
})
141-
142-
143-
}
144-
);
145-
146-
}
119+
});
120+
});
121+
}

0 commit comments

Comments
 (0)