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

Fix the icon to fit within the row in the main window tracker column. #440

Closed
wants to merge 1 commit into from
Closed

Conversation

cross
Copy link
Contributor

@cross cross commented Dec 29, 2023

For me at least, Safari on Mac OS X, the tracker icon significantly overflows in the Tracker column of the torrent list. (It does show the correct size in the Trackers filter, though. Different CSS.)

This change causes it to constrain down to the height of the column and display correctly.

@cross
Copy link
Contributor Author

cross commented Dec 29, 2023

Deluge-web-overlarge-icons Deluge-web-contained-icons

Before and after screen-shots.

@cross
Copy link
Contributor Author

cross commented Jan 7, 2024

Hey folks. Anyone available to kick this through the needed workflows?

@cas-- cas-- closed this in d9ef65d Jan 21, 2024
@cross cross deleted the icon-constrain branch February 4, 2024 21:31
doadin pushed a commit to doadin/deluge that referenced this pull request Jun 4, 2024
For me at least, Safari on Mac OS X, the tracker icon significantly
overflows in the Tracker column of the torrent list. (It does show the
correct size in the Trackers filter, though. Different CSS.)

This change causes it to constrain down to the height of the column and
display correctly.

Closes: deluge-torrent#440
doadin pushed a commit to doadin/deluge that referenced this pull request Sep 21, 2024
For me at least, Safari on Mac OS X, the tracker icon significantly
overflows in the Tracker column of the torrent list. (It does show the
correct size in the Trackers filter, though. Different CSS.)

This change causes it to constrain down to the height of the column and
display correctly.

Closes: deluge-torrent#440
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.

1 participant