-
Notifications
You must be signed in to change notification settings - Fork 14
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
j) Repeated Code in Feature Cards #21
Comments
@phipsae I'm proposing this issue! |
Since the whole page will be changed in #2 it doesn't make so much sense in my perspective (sorry). |
@aliveevie Thanks for the suggestion! I agree with Philip, we usually remove this part of the homepage. |
Okay Noted sir
Noted, I'm thinking again |
@aliveevie the issue should be more about implementing new features, so why not contributing to #2? |
Assigned it to me, issue |
Pls talk in the chat because someone is already working on that. And align, ask where you can support. |
Okay! |
Description
In the
page.tsx
under/packages/nextjs/app/page.tsx
file, the two feature cards (Debug Contracts and Block Explorer) share nearly identical structures but are hardcoded separately. This results in code duplication, making it harder to maintain and update the UI consistently.Each card consists of:
BugAntIcon
orMagnifyingGlassIcon
).This approach violates the DRY (Don't Repeat Yourself) principle. If we need to update the design, style, or structure of the cards in the future, we would have to modify multiple places in the code instead of a single, centralized component.
Goal
The goal of this refactor is to:
The text was updated successfully, but these errors were encountered: