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

feat: added and fixed colors, added wallpaper #9

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

kremzli
Copy link
Contributor

@kremzli kremzli commented Nov 21, 2024

Added new (or unused) theme options (sent and received reply, and another accent variant) and fixed a color overlap i noticed on the button to go back to latest message in a chat.
Added the cats wallpaper with a 1.5 scale and the crust accent.
Attempt number 3, whiskers included.

Added new (or unused) theme options (sent and received reply, and another accent variant colors) and fixed a color overlap i noticed on the button to go back to latest message in a chat.
@kremzli
Copy link
Contributor Author

kremzli commented Nov 21, 2024

Also if you enable dev options you can directly edit the hex colors in the app

@kremzli kremzli changed the title Added more colors and fixed color overlap Added and fixed colors, added wallpaper Dec 10, 2024
@kremzli
Copy link
Contributor Author

kremzli commented Dec 10, 2024

These are not conventional commits, so maybe squash them

@iruzo
Copy link
Member

iruzo commented Dec 10, 2024

Hey @kremzli,

First of all, thanks for the PR. You say that you "added" the "cats" wallpaper, but I don't see any image between the files in this PR. Is that wallpaper already in the app as a default wallpaper?.

Thanks in advance.

@kremzli
Copy link
Contributor Author

kremzli commented Dec 10, 2024

Yes it is included in the app

@kremzli kremzli mentioned this pull request Dec 11, 2024
@kremzli kremzli changed the title Added and fixed colors, added wallpaper feat: added and fixed colors, added wallpaper Dec 11, 2024
@iruzo
Copy link
Member

iruzo commented Dec 12, 2024

Oh I see, yeah I now know what you mean.

I'm giving this changes a try, as soon as I end testing all this and make sure we do not break anything (which might take me a little, I'm a little busy this time of the year) I will merge it.

Thanks for the PR ! <3

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