ui fixes#457
Merged
Merged
Conversation
…k items - Replace the community Slack invite with Discord (https://discord.gg/2zjBZP7yQJ) in the `failproofai --help` banner, the dashboard "Reach Us" dropdown, and the README badge (English + 14 translated READMEs); tidy a stale Slack comment in scripts/launch.ts. - Collapse the dropdown's Request a Feature / Report an Issue / Ask a Question into one "Feedback & Issues" item linking to the GitHub issue chooser. - The Slack webhook notification example is left as-is (a feature, not an invite). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
On narrow / portrait windows the policies → activity table collapsed and columns overlapped (e.g. DURATION/SESSION headers and the Event/CLI badges bled into each other). Each data cell now clips at its own edge (overflow-hidden + ellipsis, nowrap headers), the table holds a 1280px min-width and scrolls horizontally below that with a themed scrollbar, and the badge / long-header columns were widened so their content fits. Also reorders the columns to: time, decision, event, cli, tool, policy, reason, mode, duration, session. UI only — no data fetching, filters, row-expand, badges, or pagination logic changed. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
- Invite card heading 'unlock failproof perks' -> 'Share with friends'; subtitle -> 'wanna know how your friends' agents score?'. - Grammar pass over the X/LinkedIn share templates: add a missing article, 'runs local' -> 'runs locally' (adverb), fix one mis-coordinated 'and', a comma splice, and 'stops you trying' -> 'stops you from trying'. No emojis, no new content, CTAs/handles unchanged. UI copy only — no logic changes. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughThis release updates community links from Slack to Discord, restructures the ReachDevelopers dropdown, reorders the activity table, and revises audit copy, templates, and release notes. ChangesDashboard, activity table, and audit copy
Discord community link migration
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 3❌ Failed checks (2 warnings, 1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
app/policies/hooks-client.tsxESLint skipped: missing config or dependency (missing-dependency). The ESLint configuration references a package that is not available in the sandbox. 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. Comment |
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@CHANGELOG.md`:
- Around line 6-14: The CHANGELOG entries are missing the required PR number
suffix on each one-line bullet. Update every bullet under the affected section
in CHANGELOG.md so each item includes its corresponding PR number at the end,
keeping the existing wording and one-line format intact.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 62f53377-874c-4705-ba97-b107558b63a6
📒 Files selected for processing (24)
CHANGELOG.mdREADME.md__tests__/components/reach-developers.test.tsxapp/audit/_components/come-back-better-section.tsxapp/audit/_components/share-templates.tsapp/globals.cssapp/policies/hooks-client.tsxbin/failproofai.mjscomponents/reach-developers.tsxdocs/i18n/README.ar.mddocs/i18n/README.de.mddocs/i18n/README.es.mddocs/i18n/README.fr.mddocs/i18n/README.he.mddocs/i18n/README.hi.mddocs/i18n/README.it.mddocs/i18n/README.ja.mddocs/i18n/README.ko.mddocs/i18n/README.pt-br.mddocs/i18n/README.ru.mddocs/i18n/README.tr.mddocs/i18n/README.vi.mddocs/i18n/README.zh.mdscripts/launch.ts
…ipping The responsive-table pass (5bccf23) widened every badge column except mode, whose pill renders the longest badge text in the table ("bypassPermissions", 17 chars) yet sat in the narrowest badge column (7%). The blanket .activity-data-row > td overflow:hidden clip then chopped it mid-word ("bypassPermis") with no ellipsis, since text-overflow can't ellipsize an inline child. - colgroup: mode 7% -> 11% (fits bypassPermissions even at the 1280px min-width), reclaimed from policy (15->13) and reason (14->12), both of which already ellipsize. Widths still sum to 100%; added a comment mapping the bare percentages to column names so the width/column pairing can't silently drift again. - ModeBadge: inline-block + max-w-full + truncate + title, so an unexpectedly long / custom mode ellipsizes inside the pill with a hover tooltip instead of a hard mid-word cut. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WF8MwSwvxthpdGV2x5hXDW
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
A batch of dashboard / audit-page UI polish plus a community-link rebrand. Everything here is UI, copy, and links only — no behavioral or logic changes.
Changes
Community links — Slack → Discord
https://discord.gg/2zjBZP7yQJ) everywhere it's user-facing: thefailproofai --helpLINKS banner, the dashboard "Reach Us" dropdown, and the README badge (English + 14 translations). The Slack webhook notification example (examples/policies-notification.js) is intentionally left as-is — it's a feature integration, not a community link."Reach Us" dropdown
/issues/new/choose).Policies → activity table (responsive + column reorder)
min-width(1280px), scrolling horizontally below that via a themed scrollbar. The badge / long-header columns (decision, event, cli, mode, duration, session) were widened so their content fits.bypassPermissions— renders in full instead of clipping mid-word, and makeModeBadgedegrade gracefully (an in-pill ellipsis + hover tooltip) for any unexpectedly long / custom mode. The earlier responsive pass widened every badge column except mode; this finishes it.Audit invite card + share copy
/auditinvite card: heading "Share with friends", subtitle "wanna know how your friends' agents score?".runs local → runs locally(adverb), one mis-coordinated "and", a comma splice, and "stops you trying → stops you from trying". No emojis, no new content; CTAs + handles unchanged.Testing
tscandeslintclean; full unit suite green (1895 tests). CHANGELOG updated under0.0.11-beta.12.🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
Bug Fixes
Documentation