Skip to content

feat: Replace landing with riso redesign - #36

Open
wikipop wants to merge 2 commits into
masterfrom
feat/riso-redesign
Open

wikipop wants to merge 2 commits into
masterfrom
feat/riso-redesign

Conversation

@wikipop

@wikipop wikipop commented Sep 30, 2026 •

Copy link
Copy Markdown
Member

Replaces the site with the riso redesign (Nuxt 4, Tailwind 4, Nuxt Ui, Resend contact form, @nustackjs/lint).

Deploy config

Build arg (/ is prerendered, so the site key is baked into the HTML at build time - setting it only at runtime won't work):

  • NUXT_PUBLIC_TURNSTILE_SITE_KEY

Runtime env:

  • NUXT_RESEND_API_KEY
  • NUXT_TURNSTILE_SECRET_KEY

@Norbiros please create a Turnstile widget from your Cloudflare account (the one with zerya.dev):

  • Widget mode: Invisible
  • Hostname: zerya.dev

I PROMISE THIS IS THE LAST AI SLOP PR I CLAUDE DELETED MY SKILLS AND CONFIGURATION TODAY

@wikipop
wikipop requested a review from Norbiros September 30, 2026 20:50
@wikipop
wikipop force-pushed the feat/riso-redesign branch 8 times, most recently from 9d038cb to b4229cd Compare October 2, 2026 18:29
Comment thread app/assets/css/main.css Outdated
Comment thread nuxt.config.ts Outdated
Comment thread package.json Outdated
Comment thread package.json Outdated
@wikipop
wikipop force-pushed the feat/riso-redesign branch from c7bd80e to 2b000b6 Compare October 2, 2026 19:58
Nuxt 4 + Tailwind 4 + Nuxt UI hero with a Resend-backed contact form.
Linting via @nustackjs/lint (warnings fail CI), CI runs lint/typecheck/build
on pnpm, Dockerfile ships only .output, `/` is prerendered. Favicon kept.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@wikipop
wikipop force-pushed the feat/riso-redesign branch from 2b000b6 to c878efa Compare October 2, 2026 19:59

@Sanielia Sanielia left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  1. clear up slop ai comments from code
  2. i may be wrong but i think your PR introduces this warn to bun dev
Image

Comment thread app/components/HeroSection.vue Outdated
Comment thread app/components/HeroSection.vue Outdated
Comment thread app/components/SiteNavbar.vue Outdated
Comment thread app/components/SiteNavbar.vue
Comment thread app/components/SiteNavbar.vue Outdated
Comment thread app/app.vue Outdated
Comment thread app/app.vue Outdated
Comment thread server/api/contact.post.ts Outdated
Comment thread app/components/ContactDialog.vue Outdated
Comment thread app/components/ContactDialog.vue
Invisible widget (interaction-only) via @nuxtjs/turnstile in the contact
dialog; the token is verified server-side before anything is sent to
Resend. The site key is public runtime config baked in at build time
because `/` is prerendered, hence the Dockerfile build arg.
@wikipop
wikipop force-pushed the feat/riso-redesign branch from ed6715e to 2d24b14 Compare October 2, 2026 21:47
@wikipop
wikipop requested review from Norbiros and Sanielia October 2, 2026 22:19

@Norbiros Norbiros left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Regarding this:

to muszę zobaczyć żeby wypowiedzieć, efekt sam spoko, overlaj 3/10
ale jego prościej iterować niż efekt

Just to make sure we're on the same page: this doesn't mean that I approve this as the final design. What I meant is that this is a step in the right direction, but it's not finished yet. The current version gives us something that can be iterated on and improved further.

For example, there are still issues such as:

  • Zerya© - the © looks misleading here.
  • Elements like 69°N, that have no clear meaning

There are also other design issues, for example:

Image

If you want design reviews and code reviews to be handled separately, please communicate clearly when you're asking for a design review and when you're asking only for a code review. In this case, there is no approval from me on the design side of things yet.

@wikipop

wikipop commented Oct 4, 2026

Copy link
Copy Markdown
Member Author

Regarding this:

to muszę zobaczyć żeby wypowiedzieć, efekt sam spoko, overlaj 3/10
ale jego prościej iterować niż efekt

Just to make sure we're on the same page: this doesn't mean that I approve this as the final design. What I meant is that this is a step in the right direction, but it's not finished yet. The current version gives us something that can be iterated on and improved further.

For example, there are still issues such as:

  • Zerya© - the © looks misleading here.
  • Elements like 69°N, that have no clear meaning

There are also other design issues, for example:

Image

If you want design reviews and code reviews to be handled separately, please communicate clearly when you're asking for a design review and when you're asking only for a code review. In this case, there is no approval from me on the design side of things yet.

I didn't, and still don't, want anything other than a fast merge of this PR. These designs will change.

The only thing I've been asking for, and I've communicated this clearly multiple times, is to merge the PR. We also discussed this on Discord.

Regarding your or Saniela's design concerns: I hear them, but for now I don't consider them blockers. I don't want to stall this PR over relatively minor design disagreements when this part is going to be rewritten from scratch anyway, especially the layout.

Therefore, since I currently have ownership over Zerya's design direction, I'm not accepting these concerns as reasons to block the PR.

Please merge it.

Comment thread Dockerfile

COPY . .

ARG NUXT_PUBLIC_TURNSTILE_SITE_KEY

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I am affraid that this won't work, as you need to also specify ENV

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This dont work but for a different reason. In ssr mode this var should be placed in runtime env. Build arg is required for ssg/isr. I did it already.

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