Skip to content

Conversation

@fedejeanne
Copy link
Member

Allow for the error icon to appear completely when necessary e.g. when searching for an invalid regular expression.

Before this PR

The error icon is cut on the left:

image

After this PR

image

Allow for the error icon to appear completely when necessary e.g. when
searching for an invalid regular expression.
@Wittmaxi
Copy link

How does this look without the icons?

@github-actions
Copy link
Contributor

Test Results

 1 818 files  ±0   1 818 suites  ±0   1h 32m 51s ⏱️ + 2m 41s
 7 709 tests ±0   7 480 ✅  - 1  228 💤 ±0  1 ❌ +1 
24 288 runs  ±0  23 540 ✅  - 1  747 💤 ±0  1 ❌ +1 

For more details on these failures, see this check.

Results for commit a9746d2. ± Comparison against base commit 7587ea1.

@fedejeanne
Copy link
Member Author

How does this look without the icons?

Like this:

image

Why do you ask? Did I miss anything?

@fedejeanne
Copy link
Member Author

Failing test is unrelated #195

@fedejeanne fedejeanne merged commit 3ffdfe3 into eclipse-platform:master Oct 15, 2024
9 of 13 checks passed
@HeikoKlare
Copy link
Contributor

Like this:

image

Now we have quite a large whitespace to the left of the input field, just because of the (seldom) use case of having an invalid regex entered. Isn't there any other solution to the problem than this tweak?

@fedejeanne
Copy link
Member Author

LGTM

image

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.

3 participants