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

Table Panel: Fix image of image cell overflowing table cell and cells ignoring text alignment setting when a data link is added #59392

Merged
merged 4 commits into from Jan 17, 2023

Conversation

oscarkilhed
Copy link
Contributor

@oscarkilhed oscarkilhed commented Nov 28, 2022

What is this feature?
Fixes an issue where the image is overflowing the table cell when a single data link is added.
Also fixes text alignment not working on data link cells.
Reverts #55129 and moves the solution to the gauge panel.

image

Which issue(s) does this PR fix?:
Fixes #59391
Fixes #57572

@oscarkilhed oscarkilhed changed the title TablePanel: fix image of image cell overflowing table cell when a data link is added Table Panel: fixes image of image cell overflowing table cell when a data link is added Nov 28, 2022
@oscarkilhed oscarkilhed changed the title Table Panel: fixes image of image cell overflowing table cell when a data link is added Table Panel: Fix image of image cell overflowing table cell when a data link is added. Nov 28, 2022
@grafanabot grafanabot removed this from the 9.3.0 milestone Nov 30, 2022
@grafanabot
Copy link
Contributor

This pull request was removed from the 9.3.0 milestone because 9.3.0 is currently being released.

Copy link
Contributor

@JoaoSilvaGrafana JoaoSilvaGrafana left a comment

Choose a reason for hiding this comment

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

lgtm! 👏

Copy link
Member

@zoltanbedi zoltanbedi left a comment

Choose a reason for hiding this comment

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

I think the issue you opened @oscarkilhed is the same as #57572 can you please instead try to fix that issue. This pr works for the images but not for others.

@grafanabot
Copy link
Contributor

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 2 weeks if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@grafanabot grafanabot added the stale Issue with no recent activity label Dec 31, 2022
@grafanabot grafanabot added pr/external This PR is from external contributor and removed stale Issue with no recent activity labels Jan 10, 2023
@oscarkilhed oscarkilhed added this to the 9.3.4 milestone Jan 16, 2023
@oscarkilhed
Copy link
Contributor Author

@zoltanbedi this should fix both issues!

Copy link
Member

@zoltanbedi zoltanbedi left a comment

Choose a reason for hiding this comment

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

Awesome! thanks for fixing the other issue as well.

@oscarkilhed oscarkilhed removed the pr/external This PR is from external contributor label Jan 17, 2023
@oscarkilhed oscarkilhed merged commit 004705a into main Jan 17, 2023
@oscarkilhed oscarkilhed deleted the oscark/fix-image-link-overflow branch January 17, 2023 13:06
@oscarkilhed oscarkilhed changed the title Table Panel: Fix image of image cell overflowing table cell when a data link is added. Table Panel: Fix image of image cell overflowing table cell and cells ignoring text alignment setting when a data link is added Jan 17, 2023
grafanabot pushed a commit that referenced this pull request Jan 17, 2023
…ta link is added. (#59392)

* TablePanel: fix image of image cell overflowing table cell when a data link is added

* Fix image cell and gauge panel in respective places instead

(cherry picked from commit 004705a)
oscarkilhed added a commit that referenced this pull request Mar 21, 2023
…when a data link is added. (#61617)

Table Panel: Fix image of image cell overflowing table cell when a data link is added. (#59392)

* TablePanel: fix image of image cell overflowing table cell when a data link is added

* Fix image cell and gauge panel in respective places instead

(cherry picked from commit 004705a)

Co-authored-by: Oscar Kilhed <oscar.kilhed@grafana.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants