Skip to content

Fix the color of the text input box#40

Merged
andrewtavis merged 4 commits intoscribe-org:mainfrom
henrikth93:guifix
Jan 11, 2026
Merged

Fix the color of the text input box#40
andrewtavis merged 4 commits intoscribe-org:mainfrom
henrikth93:guifix

Conversation

@henrikth93
Copy link
Copy Markdown
Member

@henrikth93 henrikth93 commented Jan 5, 2026

Contributor checklist


Description

Related issue

This commit will change the color of the input box depending on theme
Will update style.rs
@henrikth93 henrikth93 requested review from RyanPaulGannon and andrewtavis and removed request for RyanPaulGannon January 5, 2026 23:38
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 5, 2026

Thank you for the pull request! 💙

The Scribe-Desktop team will do our best to address your contribution as soon as we can. If you're not already a member of our public Matrix community, please consider joining! We'd suggest that you use the Element client as well as Element X for a mobile app, and definitely join the General and Desktop rooms once you're in. Also consider attending our bi-weekly Saturday dev syncs. It'd be great to meet you 😊

Note

Scribe uses Conventional Comments in reviews to make sure that communication is as clear as possible.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 5, 2026

Maintainer Checklist

The following is a checklist for maintainers to make sure this process goes as well as possible. Feel free to address the points below yourself in further commits if you realize that actions are needed :)

  • The CHANGELOG has been updated with a description of the changes for the upcoming release and the corresponding issue (if necessary)

@andrewtavis
Copy link
Copy Markdown
Member

Do you want to try to do the Iced update before we do the review, @henrikth93? Happy to do it before if you'd prefer :)

This commit will update iced to 0.13 and fix the color of the text in the input box.
@andrewtavis
Copy link
Copy Markdown
Member

Plan is that I'll be reviewing this this weekend :) Let's be sure to update the screenshots as well 😊

Copy link
Copy Markdown
Member

@andrewtavis andrewtavis left a comment

Choose a reason for hiding this comment

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

praise: Looking really great, @henrikth93! Thanks so much for the work here to finalize these issues! 😊

note: I updated the cargo.lock dependencies as the project wasn't building for me, but now all's good :)

@andrewtavis andrewtavis merged commit 4cc2cf9 into scribe-org:main Jan 11, 2026
1 check passed
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.

Update iced to latest version Match GUI UI with Figma designs

2 participants