Fix style issues in Contract detail page

[Pabl0cks]

Description

• Fixes Some styles issues #54
• Fix zero inputs send button positioning (was overlaping close icon)

Copy button doesn't have hover effect (minor)

Haven't fixed this one because I think it means changing <Address> component. Can add it a hover:opacity-70 effect, but maybe we should do it in SE-2 then.

If you find any other style issue, lmk to fix! 🙌♥

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
abi-ninja-v2	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Feb 8, 2024 11:20am

[rin-st]

Looks great, thank you!

Regarding mobile menu burger/close button, I meant they should be the same, changing bg on hover. But I forgot that no one sees hover on mobiles, so that hover effects doesn't matter 🤷‍♂️ .

[portdeveloper]

I see no issues as well @Pabl0cks ! Great work 👍

[technophile-04]

This is looking great !!

Just an idea that came to my mind :

maybe in placeholder should we add "address or ENS"?

Just to be clear with new people they can directly use ENS too

We can just do :

const addressInputProps = { ...inputProps, placeholder: "address or ENS" };
return <AddressInput {...addressInputProps} />;

Here :

abi.ninja/packages/nextjs/components/scaffold-eth/Contract/ContractInput.tsx

Line 35 in 369826e

return <AddressInput {...inputProps} />;

What do you all think?

[Pabl0cks]

maybe in placeholder should we add "address or ENS"?

Just to be clear with new people they can directly use ENS too

Love that!! At some point I had the same thought, but forgot! I think BG crew would put ENS instinctively but external users can make use of that little help.

Pushing that change 🙌

[technophile-04]

Tysm Pablo !!