Skip to content

Conversation

@dantovska
Copy link
Collaborator

Fix the result table for the FT.SEARCH command to display text properly when it's longer

  1. Fix the cell text instead of hiding the overflow, to show an ellipsis and a copy button
  2. Fix the cell text popover content to show on the left and if bigger than the container, to apply multiline ellipsis

Note: In order to test this locally yarn build:statics must be ran

It can be tested on the sample bycicle data by running this command:
FT.SEARCH idx:smpl_bicycle "@brand:Trek"

Before After
image image
@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2025

Code Coverage - Frontend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 82.1% 20029/24397
🟡 Branches 67.43% 8662/12845
🟡 Functions 76.18% 5328/6994
🟢 Lines 82.52% 19610/23763

Test suite run success

5162 tests passing in 677 suites.

Report generated by 🧪jest coverage report action from 8963654

Copy link
Collaborator

@ArtemHoruzhenko ArtemHoruzhenko left a comment

Choose a reason for hiding this comment

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

You've picked the best option. Well done!

@dantovska dantovska merged commit 0295d45 into main Oct 8, 2025
26 of 27 checks passed
@dantovska dantovska deleted the fe/bugfix/RI-7608-ft-search-text-ellipsis branch October 8, 2025 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants