Initial modernization of the site with styling, fonts and layout#21
Open
erka wants to merge 1 commit into
Open
Initial modernization of the site with styling, fonts and layout#21erka wants to merge 1 commit into
erka wants to merge 1 commit into
Conversation
kpumuk
reviewed
May 2, 2026
kpumuk
reviewed
May 2, 2026
erka
force-pushed
the
redesign
branch
12 times, most recently
from
1eaa609 to
f1f3007
Compare
Member
|
I haven't looked at all the code changes, but the screenshots look great! |
CJCombrink
reviewed
May 18, 2026
| @@ -8,15 +8,14 @@ | |||
| - [ "(all)", "Docker", "https://hub.docker.com/_/thrift/", "Dockerfile", "", "thrift compiler in /usr/local/bin/thrift" ] | |||
| - [ "C (glib)", "", "", "", "", "language has no package manager" ] | |||
| - [ "C++", "", "", "", "", "see THRIFT-4800" ] | |||
There was a problem hiding this comment.
See my comment on THRIFT-4800 as mentioned in the "Remarks" column.
I don't think the change belongs in this PR though, but not my call to make
Member
|
+1 nice |
|
LGTM |
Member
|
Let's merge, deploy on staging and let everyone play with the new website for a little while? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
First step aligning with main ASF site design.
Changes
New Design
Layout & Components
Content & Assets
Removed
Index page
About page
Download page
Libraries page
Libraries/Java page
