Skip to content

Conversation

Hephaest
Copy link
Contributor

@Hephaest Hephaest commented Jun 28, 2025

Backgrounds

Hi there,

I found the current documentation structure is kinda confusing. As a web browser developer, I naturally navigated to the Developer section when looking for build instructions:
image

However, the building guideline is apparently not in here:
image

Eventually, I discovered the Building Guidelines are located under Getting Started — a section I thought was intended for general users wanting to explore the browser, rather than developers looking to build it from source 😂:
image

Although the homepage does include an entry point to the guide, I feel several developers have still overlooked it so they opened the issue on the GitHub...

Purpose

Given that, this PR is to help developers find the building guidelines easily (and with a minimal redirect fallback solution). The demo is shown below ⬇️

Screen.Recording.2025-06-28.at.13.49.29.mov

This redirection is simple, effective, and fully accessible.

Screen.Recording.2025-06-28.at.13.59.46.mov

Notes to Reviewers

Please feel free to push commits or make updates directly to this PR. I’m still new to the project, so I might have missed a few things — happy to learn and improve! 🙌🏼

export default function BuildingRedirect() {
return (
<>
<meta httpEquiv="refresh" content="2;url=/contribute/desktop" />
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@list-jonas
Copy link
Contributor

list-jonas commented Jul 1, 2025

Great work on the PR! I actually considered a similar change myself. From a technical perspective, the only thing I'd suggest is moving the guides directory to be a descendant of (docs) to show the navigation, and perhaps a descendant of (redirects) or something similar for better organization and to allow for more additions in the future.

I'll probably also refactor the redirect into a component at some point.

@Hephaest
Copy link
Contributor Author

Hephaest commented Jul 1, 2025

Haha Thanks! @list-jonas

Yeah, I agree with you but that will be a huge code change 😂 I’ll hand the refactor over to you folks, since my bandwidth and context are limited.

If you want to tweak my PR and enhance the handling, just go ahead. I'm totally OK!

Or if I can just leave a TODO comment before merging the PR, just let me know 😊

@IAmJafeth IAmJafeth merged commit e6c6582 into zen-browser:main Jul 16, 2025
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.

3 participants