Skip to content

Add the final Project goals 2025h2 update#1853

Merged
ehuss merged 5 commits into
rust-lang:mainfrom
nxsaken:pm/2025h2-final
May 18, 2026
Merged

Add the final Project goals 2025h2 update#1853
ehuss merged 5 commits into
rust-lang:mainfrom
nxsaken:pm/2025h2-final

Conversation

@nxsaken
Copy link
Copy Markdown
Contributor

@nxsaken nxsaken commented May 12, 2026

It's far from perfect (especially the rendering of quotes and lists), but I think it's an upgrade from the huge cards we had in the previous ones.

cc @tomassedovic

Rendered

@Kobzol
Copy link
Copy Markdown
Member

Kobzol commented May 12, 2026

As a bit of feedback, I would hide the individual comments by default, e.g. in a <details> section. It's quite hard to orient within the blog post when I have to scroll 4 pages to move between two goals because there is a long comment in the middle. I think that what would also help the blog post is to have an index of all the goals at the very beginning, so that people can quickly go to what seems interesting to them, like we do in the GSoC posts.

@tomassedovic tomassedovic mentioned this pull request May 13, 2026
@tomassedovic
Copy link
Copy Markdown
Contributor

Nice! I agree with @Kobzol's comments on collapsing comments by default and adding an index if that's not too much.

I'd love to do something about the general styling of the page (whole blog), but that's outside of the scope of this (and something we could talk about at the all hands next week too).

Comment thread content/project-goals-2026-04.md Outdated
Copy link
Copy Markdown
Contributor

@tomassedovic tomassedovic left a comment

Choose a reason for hiding this comment

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

Thanks! The ToC looks great and the page is much more legible with the comments collapsed.

I'd prefer to have all comments behind a single <details> section (see my comment), but I'm fine publishing this and just making sure we do that for any future posts.

View changes since this review

@nxsaken
Copy link
Copy Markdown
Contributor Author

nxsaken commented May 14, 2026

I did that initially and liked this level of granularity more. One reason is that it seemed less overwhelming for the reader (gradually reveals information, allows opening a comment in a new tab without expanding it, etc.). Maybe I overthought it.

@tomassedovic
Copy link
Copy Markdown
Contributor

Yeah to be fair when I saw what you did, I loved the way the page looked. But then I started clicking each detail to expand the comments and felt that's too much clicking...

I'm happy to merge as is if you prefer.

@nxsaken
Copy link
Copy Markdown
Contributor Author

nxsaken commented May 14, 2026

That's fair 😅 I'll change it in a bit.

@tomassedovic
Copy link
Copy Markdown
Contributor

Perfect, thank you!

I can't merge this, but it has my approval and it's ready to go as far as I'm concerned.

Copy link
Copy Markdown
Contributor

@ehuss ehuss left a comment

Choose a reason for hiding this comment

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

@ehuss ehuss merged commit af4cde5 into rust-lang:main May 18, 2026
6 checks passed
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.

4 participants