Skip to content

Fix inconsistent add torrent dialog window width #2975

Merged
xirvik merged 2 commits into
Novik:developfrom
drrako:fix-input-torrent-width
May 3, 2026
Merged

Fix inconsistent add torrent dialog window width #2975
xirvik merged 2 commits into
Novik:developfrom
drrako:fix-input-torrent-width

Conversation

@drrako
Copy link
Copy Markdown
Contributor

@drrako drrako commented Sep 17, 2025

Overview

This PR fixes a minor issue in the add torrent dialog, where its size changes depending on the selected input torrent file (especially in Firefox). Some long torrent file names with many space breaks can significantly stretch the modal window.

I've also updated styles so this window always take 95% of the viewport on a smaller screens.

@drrako drrako changed the title Fix inconsistant add torrent dialog width Fix inconsistent add torrent dialog window width Sep 17, 2025
@allixx
Copy link
Copy Markdown
Contributor

allixx commented Oct 10, 2025

JFYI, pull request should probably be based on develop branch.
On 2nd thought, I am really not sure if that's a bugfix or a feature, so I'll just shut up :)

@drrako
Copy link
Copy Markdown
Contributor Author

drrako commented Oct 12, 2025

@allixx I considered it a simple enough bug fix for the master branch.

@xirvik
Copy link
Copy Markdown
Collaborator

xirvik commented Apr 14, 2026

@drrako can you submit over develop? I'm testing everything there, even if it's a trivial thing that makes my life a bit easier.

@drrako drrako changed the base branch from master to develop April 29, 2026 04:12
@drrako
Copy link
Copy Markdown
Contributor Author

drrako commented Apr 29, 2026

@xirvik It is rebased against the develop now.

@xirvik xirvik merged commit b424e0a into Novik:develop May 3, 2026
xirvik added a commit to xirvik/ruTorrent that referenced this pull request May 3, 2026
Use width instead of max-width/min-width so dialog has consistent size
at each breakpoint instead of fluctuating with content.

Cherry-pick of upstream PR Novik#2975 by @drrako.
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.

3 participants