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

Improve IRC layout for SchildiChat: fix placement of capital letters inside read receipts #5

Merged
merged 1 commit into from
Nov 29, 2024

Conversation

luixxiul
Copy link

@luixxiul luixxiul commented Nov 28, 2024

This is a follow-up to 18578fe (#4)

Before After
before after

Since I'm not familiar with the workflow yet, I've created this PR against the branch for the latest release. Please let me know if there is anything I need to do.


  • I agree to release my changes under this project's license

…inside read receipts

This is a follow-up to 18578fe.

Signed-off-by: Suguru Hirahara <[email protected]>
@luixxiul luixxiul mentioned this pull request Nov 28, 2024
1 task
@luixxiul luixxiul marked this pull request as ready for review November 28, 2024 15:10
@SpiritCroc
Copy link
Member

Since I'm not familiar with the workflow yet, I've created this PR against the branch for the latest release. Please let me know if there is anything I need to do.

That's the correct thing to do, nothing else on your plate 👍

FYI I'll probably squash the commit with your existing patch for the next rebase (of course while maintaining authorship), to keep efforts at a minimum level. But that's something for us (the Schildi core team, who does upstream rebases) to do, so you don't need to worry about it.

@luixxiul
Copy link
Author

That's awesome! Thank you 😄

@SpiritCroc SpiritCroc merged commit 1ff5a6c into SchildiChat:sc_v1.11.86 Nov 29, 2024
8 of 12 checks passed
SpiritCroc added a commit to SchildiChat/schildichat-desktop that referenced this pull request Nov 29, 2024
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.

2 participants