Skip to content

Fixes overlapping text and makes styling consistent when a filter-linked row is unavailable due to ACLs#2056

Merged
Spoffy merged 1 commit intomainfrom
spoffy/fix-overlapping-text-in-filter-linked-widgets
Jan 20, 2026
Merged

Fixes overlapping text and makes styling consistent when a filter-linked row is unavailable due to ACLs#2056
Spoffy merged 1 commit intomainfrom
spoffy/fix-overlapping-text-in-filter-linked-widgets

Conversation

@Spoffy
Copy link
Contributor

@Spoffy Spoffy commented Jan 15, 2026

Context

When any widget is filter-linked to another, it only shows records connected to the one selected in the linked widget. If no row is selected, it shows "No row selected in ${X}".

When a card widget is linked to another, it only shows a single record connected to the one selected in the linked widget. If that record is filtered out the ACLs, it shows "This row is unavailable or does not exist". This also occurs in any situation where a view's cursor ends up set to null.

If these two scenarios occur at the same time, the text overlaps:
image

Proposed solution

This changes the logic for "This row is unavailable or does not exist" to only show if the widget isn't hidden due to filter linking.

Separately, this standardizes the styling for "This row is unavailable or does not exist" and "No row selected in ${X}".

Has this been tested?

  • 👍 yes, I added tests to the test suite
  • 💭 no, because this PR is a draft and still needs work
  • 🙅 no, because this is not relevant here
  • 🙋 no, because I need help

Screenshots / Screencasts

image image

Copy link
Member

@paulfitz paulfitz left a comment

Choose a reason for hiding this comment

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

Thanks @Spoffy !

@Spoffy Spoffy force-pushed the spoffy/fix-overlapping-text-in-filter-linked-widgets branch from 38c38f8 to 34e08e6 Compare January 20, 2026 14:08
@Spoffy Spoffy merged commit 68aab1c into main Jan 20, 2026
15 checks passed
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