Skip to content
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

Make default banner more palatable #5676

Merged
merged 1 commit into from
Oct 10, 2024
Merged

Make default banner more palatable #5676

merged 1 commit into from
Oct 10, 2024

Conversation

estrattonbailey
Copy link
Member

With our avatar generation (even without) this blue default banner color is pretty garish. This tones it back to look a little nicer out of the gate.

CleanShot 2024-10-10 at 12 10 28@2x
CleanShot 2024-10-10 at 12 10 26@2x

Before
CleanShot 2024-10-10 at 12 10 38@2x
CleanShot 2024-10-10 at 12 10 40@2x

@arcalinea arcalinea temporarily deployed to default-banner - social-app PR #5676 October 10, 2024 17:12 — with Render Destroyed
Copy link
Collaborator

@pfrazee pfrazee left a comment

Choose a reason for hiding this comment

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

Sure I'm +1. What's it look like in light mode?

Copy link

Old size New size Diff
7.89 MB 7.89 MB -30 B (-0.00%)

Copy link
Member

@mozzius mozzius left a comment

Choose a reason for hiding this comment

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

yessss

@estrattonbailey
Copy link
Member Author

Good call good call
CleanShot 2024-10-10 at 15 34 16@2x
CleanShot 2024-10-10 at 15 34 23@2x

@estrattonbailey
Copy link
Member Author

Additional work on the Edit Profile dialog can happen in Sam's other PR imo

@estrattonbailey estrattonbailey merged commit c4a2958 into main Oct 10, 2024
6 checks passed
@pfrazee pfrazee deleted the default-banner branch October 10, 2024 20:40
@layeredstrange
Copy link

On a first load of a new account, you could display an image in the header with instructions on changing the header as a lot of peeps obviously don't know how or choose not to. Something like this.
Banner Test 1400x440

The image included doesn't line up perfectly as I still don't know the precise resolution of the profile header.

@estrattonbailey
Copy link
Member Author

@layeredstrange that's a good idea, we'll discuss internally 👍

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.

5 participants