Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Transformations: Fix group by field transformation field name text-overflow #74173

Merged
merged 1 commit into from
Aug 31, 2023

Conversation

oscarkilhed
Copy link
Contributor

@oscarkilhed oscarkilhed commented Aug 31, 2023

What is this feature?

This fixes an issue where a very long field name would be hard to read due to text overflowing.
Before:
image

After:
image

In the event that there is a very long field name, the label will grow to accommodate the whole name. Optimally the other labels would grow to the same length as the longest field, but this would require a new solution to how we tackle forms. Since this problem should be an outlier, this solution seems good enough for now.

Fixes: https://github.com/grafana/support-escalations/issues/7266

Special notes for your reviewer:

Please check that:

  • It works as expected from a user's perspective.

@oscarkilhed oscarkilhed requested a review from a team as a code owner August 31, 2023 11:47
@grafana-delivery-bot grafana-delivery-bot bot added this to the 10.2.x milestone Aug 31, 2023
@oscarkilhed oscarkilhed changed the title Transformations: Grow field name label Transformations: Fix group by field transformation field name text-overflow Aug 31, 2023
Copy link
Contributor

@mdvictor mdvictor left a comment

Choose a reason for hiding this comment

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

LGTM

@oscarkilhed oscarkilhed merged commit de25588 into main Aug 31, 2023
34 checks passed
@oscarkilhed oscarkilhed deleted the oscark/fix-group-by-transform-text-overflow branch August 31, 2023 13:15
chauchausoup pushed a commit to chauchausoup/grafana that referenced this pull request Sep 15, 2023
@zerok zerok modified the milestones: 10.2.x, 10.2.0 Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants