Skip to content

feat(web): add ironclaw docs link#2398

Merged
think-in-universe merged 7 commits into
stagingfrom
feat/docs-link-button
Apr 14, 2026
Merged

feat(web): add ironclaw docs link#2398
think-in-universe merged 7 commits into
stagingfrom
feat/docs-link-button

Conversation

@italic-jinxin
Copy link
Copy Markdown
Contributor

@italic-jinxin italic-jinxin commented Apr 13, 2026

Summary

add Ironclaw doc link https://docs.ironclaw.com/

ui

Change Type

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • CI/Infrastructure
  • Security
  • Dependencies

Linked Issue

Validation

  • cargo fmt --all -- --check
  • cargo clippy --all --benches --tests --examples --all-features -- -D warnings
  • cargo build
  • Relevant tests pass:
  • cargo test --features integration if database-backed or integration behavior changed
  • Manual testing:
  • If a coding agent was used and supports it, review-pr or pr-shepherd --fix was run before requesting review

Security Impact

Database Impact

Blast Radius

Rollback Plan

Review Follow-Through


Review track:

@italic-jinxin italic-jinxin self-assigned this Apr 13, 2026
@italic-jinxin italic-jinxin added the risk: low Changes to docs, tests, or low-risk modules label Apr 13, 2026
@github-actions github-actions Bot added size: XS < 10 changed lines (excluding docs) contributor: regular 2-5 merged PRs labels Apr 13, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds a 'Docs' link to the tab bar, including internationalization support for English, Korean, and Chinese. A review comment suggests refactoring the CSS class names to be more generic and descriptive, as using 'status-logs-btn' for a documentation link is misleading and affects maintainability.

Comment thread crates/ironclaw_gateway/static/index.html Outdated
@serrrfirat
Copy link
Copy Markdown
Collaborator

could you provide a screenshot of the UI change?

@italic-jinxin
Copy link
Copy Markdown
Contributor Author

italic-jinxin commented Apr 13, 2026

could you provide a screenshot of the UI change?

@serrrfirat added in the summary, let me know better spot to place it.

@think-in-universe
Copy link
Copy Markdown
Contributor

I prefer placing the top-right items in the order:

  • Logs -> Docs -> Language -> Theme

@github-actions github-actions Bot added size: S 10-49 changed lines and removed size: XS < 10 changed lines (excluding docs) labels Apr 13, 2026
Copy link
Copy Markdown
Contributor

@think-in-universe think-in-universe left a comment

Choose a reason for hiding this comment

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

LGTM

This was referenced Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor: regular 2-5 merged PRs risk: low Changes to docs, tests, or low-risk modules size: S 10-49 changed lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants