Skip to content

Dev - #36

Merged
kirito666coder merged 2 commits into
mainfrom
dev
Jul 8, 2026
Merged

Dev#36
kirito666coder merged 2 commits into
mainfrom
dev

Conversation

@kirito666coder

@kirito666coder kirito666coder commented Jul 8, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • New Features

    • Updated the homepage interaction so the sign-in hover state now adds a subtle blurred overlay effect for a more polished experience.
    • Expanded the project README with a live demo link, preview images, feature highlights, setup steps, architecture overview, and deployment guidance.
  • Bug Fixes

    • Improved the homepage’s initial state handling to clear any leftover hover state on load.
    • Adjusted CI typechecking for the mobile app to run from the correct project location.

…atures

- Renamed project to 'Kirito Blog' and expanded description to highlight immersive web experience and mobile app capabilities.
- Added preview section with light and dark theme images.
- Included detailed highlights of features such as 3D visuals, GitHub authentication, and admin dashboard functionalities.
- Updated architecture diagram and project structure for clarity.
- Enhanced getting started instructions and added GitHub OAuth setup details.
@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
blog-web Ready Ready Preview, Comment Jul 8, 2026 3:09pm

@kirito666coder
kirito666coder merged commit 24fcac4 into main Jul 8, 2026
3 of 4 checks passed
@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: bbdbf673-7fee-424e-b68e-26637f1ca5ea

📥 Commits

Reviewing files that changed from the base of the PR and between c2ae728 and 8c5a2af.

⛔ Files ignored due to path filters (2)
  • Demo/web-dark.png is excluded by !**/*.png
  • Demo/web-light.png is excluded by !**/*.png
📒 Files selected for processing (3)
  • .github/workflows/ci.yml
  • README.md
  • web/src/app/(root)/page.tsx

📝 Walkthrough

Walkthrough

This PR updates the CI workflow's Mobile typecheck step to run from the mobile directory, rewrites the README with new branding, architecture, setup, feature, and deployment content, and adds hover-triggered blur state/handlers to the home page component.

Changes

Home Page Hover Blur

Layer / File(s) Summary
Signin hover blur state and wiring
web/src/app/(root)/page.tsx
Adds threeDModelBlur state, a mount-time effect resetting homePageHover, a conditional blur-sm class on the overlay container, and expanded signin hover handlers toggling both states.

README and CI Documentation Update

Layer / File(s) Summary
README rebrand, architecture, and setup docs
README.md
Rebrands intro as "Kirito Blog" with demo link/preview/highlights, adds Shiki to tech stack, adds an Architecture section with Mermaid diagram and route table, expands project structure, rewrites Getting Started steps, adds GitHub OAuth setup, restructures Web Features, and updates Deployment instructions.
Mobile CI typecheck step
.github/workflows/ci.yml
Updates the "Typecheck Mobile" step to run pnpm tsc --noEmit from the mobile directory.

Estimated code review effort: 2 (Simple) | ~10 minutes

Sequence Diagram(s)

sequenceDiagram
  participant User
  participant HomeComponent
  participant OverlayContainer
  User->>HomeComponent: hover signin
  HomeComponent->>HomeComponent: set homePageHover, threeDModelBlur=true
  HomeComponent->>OverlayContainer: apply blur-sm class
  User->>HomeComponent: leave signin
  HomeComponent->>HomeComponent: reset homePageHover, threeDModelBlur=false
  HomeComponent->>OverlayContainer: remove blur-sm class
Loading
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dev

Warning

Tools execution failed with the following error:

Failed to run tools: 13 INTERNAL: Received RST_STREAM with code 2 (Internal server error)


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant