-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Compact view for torrent list #1520
base: master
Are you sure you want to change the base?
Conversation
Can you post before & after screenshots? |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
Ping – can someone post screenshots of this? In general, I'm against adding options for stuff like this. If the compact view is "better" then we should just change to using the compact view for everyone. In general, I agree that WebTorrent current takes up a bit too much space for each torrent list item, so this is probably a good PR to merge, minus the configuration button. |
@alxhotel Thanks for posting that. I like the general idea. Issues:
|
The behaviour is now the same as before when the torrent is expanded. |
@fknop Thanks for making those changes. It appears that there is a merge conflict. Can you try to resolve that? |
It should be working now. The app still has the two options, if you want to keep only one view I can probably remove the old one as well. |
Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward? |
Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward? |
Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward? |
Hi,
I implemented this small feature for a compacted list of torrents (reduce ~by half the space needed for one item in the list) via a button icon in the header.
I know this might not be a feature necessary to the core application so I'm adding this here and leaving it to discussion!
Tests / formatting may fail for now, I did not check those yet.