Can we add a newline to the end of this file?

Can you provide a source for this SVG asset? Is it from some sort of official Ethereum brand guide?

At a high level anytime there's anything visual, the more screenshots the better

Opened the SVG in sketch and it looks as I'd expect (blue bg added for visibility)

Guessing that removing the newline here wasn't intentional?

I think what we typically do in the codebase in this sort of scenario is to declare a variable (undefined by default) and then to set it if whatever condition is true.

eg

let header;
if (!onQR && !onSIWE) {
  header = <Header />;
}

It looks like here we're instead declaring and setting the variable to whatever value, and then unsetting (by setting to null) based on whatever condition.

Just a style thing I noticed, not sure it matters nor do I personally care... just figured I'd flag

I think we can cut the siwe prefix for selector names

If we do decide to keep it, might be good to style it as siweH1

Personally I don't mind this, but the team has decided against ternaries in JSX like this: https://www.notion.so/commapp/Use-ternary-conditionals-sparingly-f4ba44a10259403592a1d15440a9847e

Can we use React.useCallback() for this

Can we memoize this

i feel like this file could be a separate diff. no need to restructure things now, though

inline comments that explain what we're doing would be helpful before all these wagmi function calls

not super familiar with react. can you explain what this function is doing?