Skip to content

Conversation

@holly-cummins
Copy link
Collaborator

@holly-cummins holly-cummins commented Jul 24, 2025

This isn't a pixel-perfect match for the main site, but I think once quarkusio/quarkusio.github.io#2365 for the vertical alignment and another PR to fix the font-weight of the links is done, it will be.

It's easiest to see the footer on individual extension pages rather than scrolling through 700 extensions. :)

@holly-cummins holly-cummins force-pushed the add-commonhaus-footer branch from 5ca5a1d to 00e7c88 Compare July 24, 2025 19:01
@holly-cummins holly-cummins requested a review from Sanne July 25, 2025 15:02
@holly-cummins holly-cummins force-pushed the add-commonhaus-footer branch from 00e7c88 to f234acf Compare July 25, 2025 19:52
@Sanne
Copy link
Member

Sanne commented Jul 28, 2025

Am I supposed to check it out? I'd gladly take your PRs on trust

@holly-cummins
Copy link
Collaborator Author

Am I supposed to check it out? I'd gladly take your PRs on trust

My thinking was just to look at the preview, but the surge preview got broken in CI, and then I fixed that but didn't have time to rebase this to trigger the preview, and also didn't have time to comment on the issue to explain that the preview was missing and I didn't really expect you to read my css and mentally render it. :)

Sorry, @Sanne !

@holly-cummins holly-cummins force-pushed the add-commonhaus-footer branch from f234acf to 3535b48 Compare August 4, 2025 18:36
@github-actions
Copy link

github-actions bot commented Aug 4, 2025

🙈 The PR is closed and the preview is expired.

@Sanne
Copy link
Member

Sanne commented Aug 5, 2025

I didn't really expect you to read my css and mentally render it. :)

Glad to hear that ! :)

np, was just wondering how you prefer things

Copy link
Member

@Sanne Sanne left a comment

Choose a reason for hiding this comment

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

Looking great!

@Sanne
Copy link
Member

Sanne commented Aug 5, 2025

N.B. I don't have merge permissions on this repo, leaving that to you

@holly-cummins holly-cummins merged commit 3f856ea into quarkusio:main Aug 5, 2025
4 checks passed
@holly-cummins holly-cummins deleted the add-commonhaus-footer branch August 5, 2025 09:02
@holly-cummins holly-cummins restored the add-commonhaus-footer branch August 7, 2025 08:12
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