Skip to content
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 12 additions & 5 deletions src/components/Programmers.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ We can only feature one awesome programmer at a time.
Find comments below to help you along.
*/

import React from 'react';
import React, { useState } from 'react';

// Use this variable ONLY to initialize a slice of state!
// There is something in the JSX right now breaking this rule...
Expand All @@ -27,19 +27,26 @@ export const listOfAwesome = [
export default function Programmers() {
// We'll have to use the state hook twice, as we need two slices of state.
// The programmers list on the one hand, and the id of the featured programmer on the other.
const [ programmers, setProgrammers] = useState(listOfAwesome);
const [ featured, setFeatured] = useState(null);

const getNameOfFeatured = () => {
// Leave this for last!
// This is NOT an event handler but a helper function. See its usage inside the JSX.
// It's going to utilize both slices of state to return the _name_ of the featured dev.
// The beauty of closures is that we can "see" both slices of state from this region
// of the program, without needing to inject the information through arguments.
for(let i = 0; i < programmers.length; i++){
if (programmers[i].id === featured) {
return programmers[i].name;
}
}
};

const style = {
fontSize: '1.5em',
marginTop: '0.5em',
color: 'royalblue', // 🤔 color turns to gold, when celebrating
color: featured ? 'gold' : 'royal blue', // 🤔 color turns to gold, when celebrating
};

return (
Expand All @@ -50,9 +57,9 @@ export default function Programmers() {
/* Nasty bug! We should map over a slice of state, instead of 'listOfAwesome'.
We might think: "it works, though!" But if the list of programmers is not state,
we could never add or edit programmers in the future. The list would be a static thing." */
listOfAwesome.map(dev =>
programmers.map(dev =>
<div className='programmer' key={dev.id}>
{dev.name} <button onClick={() => { /* in here set the featured id to be dev.id */ }}>Feature</button>
{dev.name} <button onClick={() => {setFeatured(dev.id)}}>Feature</button>
</div>
)
}
Expand All @@ -62,7 +69,7 @@ export default function Programmers() {
// Ternaries are fantastic to render "one thing or the other" depending on the "truthiness" of something.
// Pseudo-code: if the currently featured id is truthy render text 1, otherwise render text 2.
// Replace the hard-coded false with the correct variable.
false
featured
? `🎉 Let's celebrate ${getNameOfFeatured()}! 🥳`
: 'Pick an awesome programmer'
}
Expand Down