-
Notifications
You must be signed in to change notification settings - Fork 2
feat: implement fork details modal with metrics and education #54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
jubalm
wants to merge
6
commits into
main
Choose a base branch
from
feature/fork-gauge
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
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
- Add Radix UI dialog component for enhanced UI patterns - Rename fork pressure terminology to fork risk - Update color scheme and styling for consistency - Add ProseCard and improved component structure - Add tw-animate-css for animation support - Update dependencies and TypeScript configurations
Add missing ForkDetailsCard and supporting components that were imported in the latest fork gauge improvements but never created. This fixes the build failure: "Could not resolve './ForkDetailsCard'". Changes: - Create ForkDetailsCard.tsx: Interactive modal dialog for fork risk details - Displays detailed fork metrics (largest bond, active disputes, threshold) - Includes fork education content with ASCII art visualization - Links to learning resources and whitepaper - Full accessibility support with proper ARIA labels - Create ForkAsciiArt.tsx: ASCII art visualization component - Responsive fork diagram with responsive sizing - Integrates with the details dialog - Create ui/dialog.tsx: Radix UI Dialog component wrapper - Accessible modal implementation following WAI-ARIA patterns - Consistent with project's UI component library - Supports keyboard navigation and focus management Fixes CI build failure and enables summary job to complete.
b7e30a9 to
30e7e13
Compare
Replace terminology to align with final Augur team specifications: - Normal → None (0% risk) - Low → Low (0-10% risk) - Moderate → Medium (10-25% risk) - High → High (25-75% risk) - Elevated → Extreme (75%+ risk) Updated across all components and utilities: - ForkGauge: Visual gauge labels - ForkBadge: Risk level styling - ForkDataProvider: Risk level calculations - ForkControls: Demo mode scenarios - demoDataGenerator: Demo data generation logic - Type definitions in gauge.ts Maintains all existing functionality while reflecting new terminology.
Changed 'None' back to 'Normal' for the 0% risk state to match Augur team's decision correctly: - Normal (0% risk) - not None - Low (0-10% risk) - Medium (10-25% risk) - High (25-75% risk) - Extreme (75%+ risk)
More accurate terminology expressing that the possibility of a fork is zero: - No Risk (0% risk) - clearly shows zero fork possibility - Low (0-10% risk) - Medium (10-25% risk) - High (25-75% risk) - Extreme (75%+ risk)
* feat: migrate fork learning pages to content collection - Create MDX content collection for fork learning pages - Add 'What is a Fork?' landing page with fork basics - Add 'How Disputes & Bonds Work' with mechanics explanation - Add 'What To Do at Different Risk Levels' action guide - Create dynamic route handler for fork content pages - Convert HTML tables to markdown syntax for readability * feat: add fork learning layout and navigation components - Create LearnLayout for fork education pages with prose styling - Add LearnNavigation component for topic navigation - Add ProseCard component for styled content blocks - Add blockquote styling with muted-foreground border - Style prose code blocks with transparent backgrounds - Update ordered list marker colors * refactor: consolidate fork learning content cleanup and updates - Update astro.config.mjs for content collection support - Update package dependencies for MDX content - Clean up obsolete fork pages (bonds, calculation, trigger, etc.) - Reorganize fork learning content for improved readability - Consolidate fork learning materials reducing repetition * refactor: replace 'vote' terminology with 'choose', 'migrate', and 'govern' Update fork learning content to use more precise language: - 'choose' for fork decision-making contexts - 'migrate' when describing REP migration action - 'govern' for diagram context * refactor: replace "market outcome" with "outcome" in fork learning docs
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
Implements interactive fork risk details modal, enabling users to view detailed metrics, understand fork mechanics, and access educational resources. This completes the fork gauge feature infrastructure with a comprehensive modal dialog showing real-time fork risk data alongside educational content.
Features
Fork Details Modal
Technical Details
Related