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

#1441 Fix input background in Firmware Updater dialog #1465

Merged
merged 1 commit into from
Sep 20, 2022

Conversation

francescospissu
Copy link
Contributor

Motivation

In the Firmware Updater dialog the "Select a board..." input has a contrast issue. See details here.

Change description

Changed the background of the input accordingly with the selected theme.

Dark:

Screenshot 2022-09-19 120427

Screenshot 2022-09-19 120538

Screenshot 2022-09-19 120613

Light:
Screenshot 2022-09-19 144537

Screenshot 2022-09-19 144607

Screenshot 2022-09-19 144647

Other information

Closes #1441.

Reviewer checklist

  • PR addresses a single concern.
  • The PR has no duplicates (please search among the Pull Requests before creating one)
  • PR title and description are properly filled.
  • Docs have been added / updated (for bug fixes / features)

@francescospissu francescospissu added topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project labels Sep 19, 2022
@francescospissu francescospissu self-assigned this Sep 19, 2022
Copy link
Contributor

@AlbyIanna AlbyIanna left a comment

Choose a reason for hiding this comment

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

LGTM 🍕

@francescospissu francescospissu merged commit b3ab5cb into main Sep 20, 2022
@francescospissu francescospissu deleted the fspissu/fw-uploader-bg branch September 20, 2022 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Dark theme] Text contrast for selected board in Firmware Updater needs fixing
2 participants