Repository navigation
fix(ActionButton): add hover and pressed color states for icon in input fields - #724
Merged
18202781743 merged 1 commit intoOct 16, 2026
Conversation
…ut fields - hover: icon color becomes opaque foreground (alpha 1.0) - pressed: icon color becomes active highlight color - palette.windowText now always bound to ColorSelector.textColor so the palette system picks up all states automatically - adapts to light/dark themes via the palette system
Reviewer's guide (collapsed on small PRs)Reviewer's GuideUpdates ActionButton’s palette configuration so icons in input-field controls transition from subdued in the normal state to opaque on hover and the active highlight color when pressed, with light/dark theme behavior handled by the existing palette system. State diagram for ActionButton icon color statesstateDiagram-v2
[*] --> Normal
Normal --> Hovered: pointer enters
Hovered --> Normal: pointer leaves
Hovered --> Pressed: press
Pressed --> Hovered: release
Normal: textColor.common = alpha 0.7
Hovered: textColor.common = alpha 1.0
Pressed: textColor = D.DTK.makeColor(D.Color.Highlight)
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Author
18202781743
approved these changes
Oct 16, 2026
Contributor
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 18202781743, iCancely The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
18202781743
merged commit Oct 16, 2026
984acd0
into
linuxdeepin:v25-flowstyle
18 of 20 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
palette.windowTextnow always bound toD.ColorSelector.textColorso the palette system picks up all states automatically (previously only bound on pressed, hover wasundefined)normalDark/hoveredDarkneededRationale
The
ActionButtonused insideLineEdit/PasswordEdit(clear button, echo toggle) had no icon color change on hover. This follows the same pattern already established inDS.Style.comboBox.flatText(normal 70% alpha, hovered opaque, pressed highlight).Scope
Only
qt6/src/qml/ActionButton.qmlis modified.Summary by Sourcery
Add responsive, theme-aware icon colors to ActionButton controls in input fields.
New Features:
Bug Fixes:
Enhancements: