Skip to content

fix(lxlweb): expand supersearch on pill click - #1698

Merged
jesperengstrom merged 2 commits into
developfrom
fix-lxlweb-pill-clicks
Oct 7, 2026
Merged

jesperengstrom merged 2 commits into
developfrom
fix-lxlweb-pill-clicks

Conversation

@jesperengstrom

@jesperengstrom jesperengstrom commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Description

Solves

A regression has happened (maybe as a result of implementing CM selection) where clicking on pills no longer opens supersearch (instead adding a selection to the pill). This is fatal on mobile screens where pills sometimes covers the whole input and there's no way to continue writing.

Now ignoring all events on the atomic widget (double click to edit still works).

Also updated the custom selection css to cover cases where selections mysteriously become grey.
Skärmavbild 2026-10-02 kl  13 21 52

Summary of changes

  • ignore events on qualifier widget (let clicks reach supersearch.showExpandedSearch)
  • update theme extension css
  • add test

@jesperengstrom
jesperengstrom marked this pull request as ready for review October 2, 2026 13:40
@jesperengstrom
jesperengstrom requested a review from olovy October 6, 2026 15:59

@olovy olovy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@jesperengstrom
jesperengstrom merged commit 184100d into develop Oct 7, 2026
3 checks passed
@jesperengstrom
jesperengstrom deleted the fix-lxlweb-pill-clicks branch October 7, 2026 09:51
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.

2 participants