Skip to content

Commit

Permalink
Fix new images in NavigationBar and CharacterCard
Browse files Browse the repository at this point in the history
  • Loading branch information
HugoVS26 committed Nov 3, 2023
1 parent fbc9a23 commit 31b4ba0
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 9 deletions.
6 changes: 3 additions & 3 deletions src/components/CharacterCard/CharacterCard.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ const CharacterCard = ({
<div className="character-card__icon-container">
<NavLink to={`/`}>
<img
src={"images/button-info.webp"}
src={"images/button-info.svg"}
alt="Go to character details"
width="48"
height="48"
Expand All @@ -35,7 +35,7 @@ const CharacterCard = ({
</NavLink>
<button>
<img
src={"images/button-trash.webp"}
src={"images/button-trash.svg"}
alt="Eliminate character"
width="48"
height="48"
Expand All @@ -44,7 +44,7 @@ const CharacterCard = ({
</button>
<NavLink to={`/`} end>
<img
src={"images/button-edit.webp"}
src={"images/button-edit.svg"}
alt="Go to edit character"
width="48"
height="48"
Expand Down
8 changes: 4 additions & 4 deletions src/components/NavigationBar/NavigationBar.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ describe("Given a NavigationBar component", () => {
describe("When is rendered", () => {
test("Then it should show the image `button-home`", () => {
const expectedAltText = "Home button";
const expectedImageSrc = "images/button-home.webp";
const expectedImageSrc = "images/button-home.svg";

render(
<BrowserRouter>
Expand All @@ -24,7 +24,7 @@ describe("Given a NavigationBar component", () => {

test("Then it should show the image `button-plus`", () => {
const expectedAltText = "Add character button";
const expectedImageSrc = "images/button-plus.webp";
const expectedImageSrc = "images/button-plus.svg";

render(
<BrowserRouter>
Expand All @@ -40,7 +40,7 @@ describe("Given a NavigationBar component", () => {
});

describe("When is rendered and we are at `/home`", () => {
test("Then it should show the image `button-plus.webp` with the className `navigation__icon--active`", () => {
test("Then it should show the image `button-plus.svg` with the className `navigation__icon--active`", () => {
const expectedAltText = "Home button";
const expectedImageClassName = "navigation__icon--active";

Expand All @@ -58,7 +58,7 @@ describe("Given a NavigationBar component", () => {
});

describe("When is rendered and we are at `/addCharacter`", () => {
test("Then it should show the image `button-plus.webp` with the className `navigation__icon`", () => {
test("Then it should show the image `button-plus.svg` with the className `navigation__icon`", () => {
const expectedAltText = "Home button";
const expectedImageClassName = "navigation__icon";

Expand Down
4 changes: 2 additions & 2 deletions src/components/NavigationBar/NavigationBar.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ const NavigationBar = (): React.ReactElement => {
? "navigation__icon--active"
: "navigation__icon"
}
src="images/button-home.webp"
src="images/button-home.svg"
alt="Home button"
width={48}
height={48}
Expand All @@ -32,7 +32,7 @@ const NavigationBar = (): React.ReactElement => {
? "navigation__icon--active"
: "navigation__icon"
}
src="images/button-plus.webp"
src="images/button-plus.svg"
alt="Add character button"
width={48}
height={48}
Expand Down

0 comments on commit 31b4ba0

Please sign in to comment.