Skip to content

feat: add navbar and footer #6

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.

Sign up for GitHub

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

Jump to bottom
Merged
ErwanDecoster merged 32 commits into from
May 13, 2025
Merged

feat: add navbar and footer #6

ErwanDecoster merged 32 commits into from
May 13, 2025

Conversation

@ErwanDecoster
Copy link
Member

@ErwanDecoster ErwanDecoster commented May 5, 2025 •
edited
Loading

This PR focuses solely on adding the navigation bar and footer. It does not address the protocol change or theme change; these features will be handled in a future PR.

@ErwanDecoster
style: adjust layout of preview tables for improved responsiveness
f1cf8b9
@ErwanDecoster
refactor: pass className prop to preview table components for better …
2f725d1 
…layout control
@ErwanDecoster
feat: add bellecour chain configuration and injected wallet provider.
122e547
@ErwanDecoster
feat: add user authentication and avatar selection features
5ee9127
@ErwanDecoster
fix: update favicon to use iexec logo and remove unused react.svg file
2ef9a14
@ErwanDecoster
feat: integrate WagmiProvider for improved wallet management
5507253
@ErwanDecoster
refactor: update StatusCell type definitions and improve timestamp ha…
ed945d8 
…ndling in formatElapsedTime
@ErwanDecoster
fix: move @trivago/prettier-plugin-sort-imports dependency devDepende…
ae181c8 
…ncies
@ErwanDecoster
fix: better routing, fixing some bugs
8176f3d
@ErwanDecoster
feat: add Radix UI Select component and update styles for popover
48fb59f
@ErwanDecoster
fix: remove unused QueryClient import from main.tsx
b55b5a8
@ErwanDecoster
feat: enhance TopNavBar with responsive menu, account button and prot…
2bc182e 
…ocol selector
@ErwanDecoster
feat: add IexecAccount component to TopNavBar with copy functionality
6849c32
@ErwanDecoster
style: format Button component for better readability
482ec31
@ErwanDecoster
refactor: improve layout and styling of TopNavBar component
3201a54
@ErwanDecoster
feat: rename TopNavBar component to Navbar and better responsive
64afbff
@ErwanDecoster
fix: update responsive classes in NavBar for better future layout
cb8beb0
@ErwanDecoster ErwanDecoster self-assigned this May 5, 2025
@ErwanDecoster ErwanDecoster marked this pull request as ready for review May 6, 2025 06:32
PierreJeanjacquot
PierreJeanjacquot reviewed May 6, 2025
View reviewed changes
Copy link
Member

@PierreJeanjacquot PierreJeanjacquot left a comment

Choose a reason for hiding this comment

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

see comments

PierreJeanjacquot
PierreJeanjacquot requested changes May 7, 2025
View reviewed changes
Copy link
Member

@PierreJeanjacquot PierreJeanjacquot left a comment

Choose a reason for hiding this comment

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

1 bug to fix

@PierreJeanjacquot PierreJeanjacquot self-requested a review May 13, 2025 08:56
PierreJeanjacquot
PierreJeanjacquot approved these changes May 13, 2025
View reviewed changes
Copy link
Member

@PierreJeanjacquot PierreJeanjacquot left a comment

Choose a reason for hiding this comment

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

LGTM

@ErwanDecoster ErwanDecoster merged commit 25d7c44 into main May 13, 2025
2 checks passed
@ErwanDecoster ErwanDecoster deleted the feature/add-navbar-and-footer branch May 13, 2025 09:08
@iexec-release-please-app iexec-release-please-app bot mentioned this pull request May 13, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PierreJeanjacquot PierreJeanjacquot PierreJeanjacquot approved these changes

Assignees

@ErwanDecoster ErwanDecoster

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ErwanDecoster @PierreJeanjacquot