IBX-12110: Fixed alert colour, UDW search ring and bookmark icons - #2007
Open
tischsoic wants to merge 6 commits into
Open
IBX-12110: Fixed alert colour, UDW search ring and bookmark icons#2007tischsoic wants to merge 6 commits into
tischsoic wants to merge 6 commits into
Conversation
tischsoic
force-pushed
the
IBX-12110-ui-consistency-fixes
branch
2 times, most recently
from
August 18, 2026 08:02
cfc566e to
bd2b46a
Compare
albozek
approved these changes
Aug 18, 2026
GrabowskiM
approved these changes
Aug 18, 2026
GrabowskiM
approved these changes
Aug 18, 2026
alekmick
approved these changes
Aug 18, 2026
barbaragr
approved these changes
Aug 20, 2026
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
$ibexa-color-complementary-100 pointed at $color-primary_alt-20 while every other step of the complementary ramp maps to the info blue, so info alerts rendered lavender with blue text and border. Regressed in ef6b8ba. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
The borderless search input already reset outline on focus but not the design system box-shadow ring, which also overlapped the magnifier button and hid its own ring. The button now stacks above the absolutely positioned input. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
The content view header showed a bookmark ribbon where QA expects a star, so it now uses favourite-outline/favourite-filled. The UDW meta preview, the UDW bookmarks tab and the main menu entry referenced 'bookmark' and 'bookmark-active', neither of which exists in the icon sprite, so they rendered empty. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
The address input wrapper is positioned, so it painted over the non-positioned button and hid the left band of its focus ring when reached with Tab. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
tischsoic
force-pushed
the
IBX-12110-ui-consistency-fixes
branch
from
August 21, 2026 08:59
bd2b46a to
f3e789d
Compare
|
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
9 participants
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.
Related PRs:
Description:
Covers QA points 2, 3, 6, 7 and 10:
$ibexa-color-complementary-100pointed at the violet$color-primary_alt-20instead of the info blue the rest of the ramp uses.outlineon focus but not the design systembox-shadowring, which also covered the magnifier button's own ring.bookmark/bookmark-active, which do not exist in the icon sprite and rendered empty in the UDW meta preview, the UDW bookmarks tab and the main menu.Point 4 (unstyled Create content search box) is fixed by IBX-12237 instead — ibexa/design-system#129. Points 8, 11 and 13 are not included — they need a design decision first.
For QA:
Documentation: