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.

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

fix: placeholder alignments #885

Merged
merged 1 commit into from
Feb 3, 2025
Merged

fix: placeholder alignments #885

merged 1 commit into from
Feb 3, 2025

Conversation

solracsf
Copy link
Member

@solracsf solracsf commented Feb 3, 2025

  • Remove subject font-size
  • Remove also on message as it is already defined (same size) on parent

Before

image

After

image

@solracsf
fix: placeholder alignments
7050346 
Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
@mwinkens
Copy link
Contributor

mwinkens commented Feb 3, 2025 •
edited
Loading

Interesting, it doesn't look broken on default on my screen with firefox, but chrome, but not to the degree in the screenshots

@nickvergessen nickvergessen merged commit 4fc7285 into main Feb 3, 2025
40 checks passed
@nickvergessen nickvergessen deleted the fixPlaceholderAlign branch February 3, 2025 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mwinkens mwinkens mwinkens approved these changes

@nickvergessen nickvergessen Awaiting requested review from nickvergessen

Assignees
No one assigned
Labels
3. to review design
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@solracsf @mwinkens @nickvergessen