Skip to content
Merged
Show file tree
Hide file tree
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
15 changes: 0 additions & 15 deletions src/components/footer.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
}
`

const Logo = styled(props => <a {...props} />)`

Check warning on line 25 in src/components/footer.js

View workflow job for this annotation

GitHub Actions / build

Anchors must have content and the content must be accessible by a screen reader
background-color: var(--navbar-background-color);
display: flex;
flex-direction: row;
Expand All @@ -30,7 +30,7 @@
justify-content: space-between;
`

const SponsorInfo = styled.div`

Check warning on line 33 in src/components/footer.js

View workflow job for this annotation

GitHub Actions / build

'SponsorInfo' is assigned a value but never used
display: flex;
flex-direction: row;
align-items: center;
Expand All @@ -47,7 +47,7 @@
}
`

const LegalText = styled(props => <a {...props} />)`

Check warning on line 50 in src/components/footer.js

View workflow job for this annotation

GitHub Actions / build

Anchors must have content and the content must be accessible by a screen reader
color: var(--footer-text-color);
text-decoration: underline;
`
Expand Down Expand Up @@ -89,21 +89,6 @@
href="https://www.commonhaus.org/trademarks/">Trademark List</LegalText>. Trademarks of third parties are owned
by their respective holders and their mention here does not suggest any endorsement or association.
</LicenseInfo>

<SponsorInfo>
Sponsored by
<Logo href="https://www.redhat.com/" style={{ width: "6rem" }}>
<StaticImage
className="logo"
placeholder="none"
backgroundColor="black"
layout="constrained"
formats={["auto", "webp", "avif"]}
src="../images/redhat_reversed.svg"
alt="Red Hat logo"
/>
</Logo>
</SponsorInfo>
</FooterBar>
)
}
Expand Down
4 changes: 2 additions & 2 deletions src/components/footer.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,8 @@ describe("footer", () => {
render(<Footer />)
})

it("renders a supported by statement", () => {
expect(screen.getByText(supportedText)).toBeTruthy()
it("does not renders a supported by statement", () => {
expect(screen.findByText(supportedText)).toBeTruthy()
})

})
11 changes: 11 additions & 0 deletions src/style.css
Original file line number Diff line number Diff line change
Expand Up @@ -171,6 +171,17 @@ html {
font-weight: var(--font-weight-normal);
}

@media screen and (max-width: 1366px) {
html {
--site-margins: 4rem;
--navbar-padding: 1.5rem;
--a-generous-space: 50px;
--a-modest-space: 12px;
--a-small-space: 6px;
--a-vsmall-space: 5px;
}
}

/** Adjustments for small screens */
@media screen and (max-width: 1024px) {
html {
Expand Down
2 changes: 1 addition & 1 deletion src/templates/extensions-added-by-year-list.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ describe("extensions added page for a year", () => {

it("renders the correct link", () => {
const links = screen.getAllByRole("link")
const link = links[links.length - 7]// Look at the last one that's not in the footer, because the top of the page will have a menu and the bottom will have footers - this is also testing the sorting
const link = links[links.length - 6]// Look at the last one that's not in the footer, because the top of the page will have a menu and the bottom will have footers - this is also testing the sorting
expect(link).toBeTruthy()
// Hardcoding the host is a bit risky but this should always be true in test environment
expect(link.href).toBe("http://localhost/ambiguous-slug")
Expand Down
2 changes: 1 addition & 1 deletion src/templates/extensions-added-list.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ describe("extensions added page", () => {

it("renders the correct link", () => {
const links = screen.getAllByRole("link")
const link = links[links.length - 7]// Look at the last one that's not in the footer, because the top of the page will have a menu and the bottom will have footers - this is also testing the sorting
const link = links[links.length - 6]// Look at the last one that's not in the footer, because the top of the page will have a menu and the bottom will have footers - this is also testing the sorting
expect(link).toBeTruthy()
// Hardcoding the host is a bit risky but this should always be true in test environment
expect(link.href).toBe("http://localhost/ambiguous-slug")
Expand Down
Loading