vinishjail97 opened a new pull request, #915:
URL: https://github.com/apache/incubator-xtable/pull/915
Part of #855.
## What is the purpose of the pull request
Add the 0.4.0-incubating release blog post to the website, and fix two layout
caps that were keeping the blog pages narrow.
The post is the last website piece of the 0.4.0 release. The release notes
page
and the downloads entries already landed in #913, so this is what remains
before
the announcement mail goes out.
## Brief change log
- Added `website/blog/apache-xtable-0.4.0-release.mdx`. It covers the three
headline items, the Spark runtime jar, the Paimon and Parquet sources, and
Delta Kernel, then links to `/releases/release-0.4.0-incubating` for the
full
change list rather than duplicating it.
- Raised `--ifm-container-width` and `--ifm-container-width-xl` to 1600px.
`BlogLayout` wraps the page in Infima's `.container`, which caps at
`--ifm-container-width`. That defaults to 1140px and nothing overrode it,
so
the blog stopped growing on any monitor wider than that.
- Moved the `width: 50%` on `BlogPostItem/Container` out of an inline style
and
into a `.blogPostCard` class. As an inline style no stylesheet could
override
it, it held at every breakpoint, and it applied to the single-post page as
well as the list, because both views render the same container. It is now
applied only when `isBlogPostPage` is false, and drops to one card per row
under 996px.
The single-post page being squeezed to half width was the more visible of the
two, and it is fixed as a side effect of the second change.
## Verify this pull request
This pull request is a website-only change with no test coverage.
Verified by running the site locally with `npm run start` in `website/` and
checking, at both wide and narrow viewports:
- `/blog/` renders two cards per row above 996px and one below it.
- `/blog/apache-xtable-0.4.0-release` renders at full column width, and its
release-notes link resolves.
- The existing blog posts and the `/releases/downloads` page still render.
## Note for reviewers
This carries two separable things: the release post, which belongs to #855,
and
the layout fix, which does not. I kept them in one PR because both are
website-only and the layout problem is most visible on the new post. If you
would rather review them apart, say so and I will split the layout change
into
its own `[MINOR]` PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
