Skip to content

Conversation

@Le-Caignec
Copy link
Collaborator

No description provided.

@Le-Caignec Le-Caignec self-assigned this Aug 6, 2025
@vercel
Copy link

vercel bot commented Aug 6, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
documentation ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 7, 2025 9:15am

This comment was marked as outdated.

@Le-Caignec Le-Caignec changed the title Feature/migrate explorer pages to component feat: migrate explorer pages to component Aug 6, 2025
@Le-Caignec Le-Caignec requested a review from Copilot August 6, 2025 19:35
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modernizes the explorer documentation pages by migrating from custom HTML/CSS to reusable Vue components. The main goals are improved maintainability, consistency, and code reuse across the documentation.

Key changes:

  • Replace custom HTML structures with standardized Vue components (ImageViewer, FeatureCard, UseCaseCard)
  • Remove extensive custom CSS in favor of component-based styling
  • Add new customizable props to existing components for better flexibility

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/overview/tooling-and-explorers/iexec-explorer.md Migrates image displays and feature cards to Vue components, removes custom CSS
src/overview/tooling-and-explorers/blockchain-explorer.md Converts network cards to UseCaseCard component usage
src/components/UseCaseCard.vue Adds customizable label props for demo and GitHub buttons
src/components/ImageViewer.vue New component for consistent image display with links and captions
src/components/FeatureCard.vue New component for feature showcase cards with hover effects

Copy link
Member

@ErwanDecoster ErwanDecoster left a comment

Choose a reason for hiding this comment

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

A few small comments and also a PR proposal.

Copy link
Member

@ErwanDecoster ErwanDecoster left a comment

Choose a reason for hiding this comment

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

Good job

@Le-Caignec Le-Caignec merged commit ce1daae into main Aug 7, 2025
3 checks passed
@Le-Caignec Le-Caignec deleted the feature/migrate-to-component branch August 7, 2025 10:24
This was referenced Aug 6, 2025
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