Skip to content

Conversation

jevenski
Copy link
Contributor

This patch removes the old drag mask object, and makes drag and drop action instant. It also separates moving directory list frames from moving the containing dialog windows for better maintenance.

Screen Recording 2024-11-24 131802

Commit message:

  • Without the drag mask object, dialog windows can be moved around instantly during dragging the mouse.
  • Move dir list frame dnd movement function to the _getdir plugin for separation.

- Without the drag mask object, dialog windows can be moved around
  instantly during dragging the mouse.
- Move dir list frame dnd movement function to the _getdir plugin
  for separation.
@stickz
Copy link
Collaborator

stickz commented Nov 25, 2024

Could we add a maximum width to the category list? So the user can't make the torrent info list-table disappear?

@jevenski
Copy link
Contributor Author

@stickz Sure thing. I think we can simply add a hard-coded max width restriction to the resizeLeft() function, say like 50% of the window width. We don't need to complicate the WebUI options with this one, as 50% should be more than enough for most use cases.

@stickz
Copy link
Collaborator

stickz commented Nov 25, 2024

Could we put a new tab in the settings menu called developer? Then put this restriction plus these two in there? It's helpful for us to be able to change these values, without having to find the hard-coded values.

dev_settings

@jevenski
Copy link
Contributor Author

How does this one look like? There is a difference in the units of min width (pixel) and max width (%). Setting a limitation on the max width using percentage should be more intuitive because max width is dependent on the width of the entire window, which is subject to change.

20241125115920

@stickz
Copy link
Collaborator

stickz commented Nov 25, 2024

How does this one look like?

Looks good.

@@ -2223,9 +2224,12 @@ var theWebUI = {
if (!w)
return;
const offcanvas = $("#offcanvas-sidepanel");
if (theWebUI.settings["webui.list_table_min_height"]) {
if (theWebUI.settings["webui.side_panel_min_width"]) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a typo fix that we can also backport to v5.1.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes, agreed! Can you submit a PR to master for this?

@stickz stickz merged commit 9575077 into Novik:develop Nov 25, 2024
@jevenski jevenski deleted the instant-dnd branch November 26, 2024 00:09
stickz added a commit to stickz/docker-rtorrent-rutorrent that referenced this pull request Apr 27, 2025
**Full Changelog**: Novik/ruTorrent@v5.1.7...v5.2.1

# ruTorrent v5.2.1
This is a critical hotfix to resolve breakage of RSS feeds. It's highly recommended to upgrade.
In addition, the white border is fixed on dark themes. A stopped state is also added to torrents.

## What's Changed
* Adding Stopped state by ranirahn in Novik/ruTorrent#2893
* Fix for broken RSS feeds break the rutorrent. by ranirahn in Novik/ruTorrent#2894
* Fix white border in dark themes by drrako in Novik/ruTorrent#2898

**Full Changelog**: Novik/ruTorrent@v5.2.0...v5.2.1

# ruTorrent v5.2.0 is now Stable
This is a release contains more mobile design improvements, performance improvements and various new features. It builds on progress made in ruTorrent `v5.1` toward mobile development. 

## Critical Improvement Summary

1. Torrents now have bigger font to make them easier to read.
2. There is no more drag mask when moving around dialog boxes.
3. Internet explorer is no longer supported. (Yes finally, we dropped it)
4. Various performance improvements to tables to improve responsiveness.
5. The clear log now has a bootstrap button in the bottom right corner.
6. Many other smaller improvements, bug fixes and new features.

## What's Changed
* Fix torrent deleted without confirmation by koblack in Novik/ruTorrent#2804
* filedrop plugin: support dropped links by pekman in Novik/ruTorrent#2806
* Remove drag mask object by jevenski in Novik/ruTorrent#2805
* Covert clear log to a floating button by jevenski in Novik/ruTorrent#2810
* Use bootstrap dropdown on recent trackers by jevenski in Novik/ruTorrent#2809
* Use flex layout on context menu items by jevenski in Novik/ruTorrent#2812
* Adjust status bar and top menu by jevenski in Novik/ruTorrent#2813
* Define dialog window colors using variables by jevenski in Novik/ruTorrent#2815
* Distribute WebUi space using flex layout by jevenski in Novik/ruTorrent#2826
* Remove Internet Explorer support, and some useragent sniffing hacks by anthonyryan1 in Novik/ruTorrent#2825
* Change various s-table methods by jevenski in Novik/ruTorrent#2821
* Define status bar icons using base64 strings by jevenski in Novik/ruTorrent#2833
* Animate category panel by jevenski in Novik/ruTorrent#2831
* Define getter functions on s-table by jevenski in Novik/ruTorrent#2830
* Use bootstrap list group on options window by jevenski in Novik/ruTorrent#2819
* Define s-table header styles using variables by jevenski in Novik/ruTorrent#2836
* Improve s-table column resizing by jevenski in Novik/ruTorrent#2837
* Control alternate row color using CSS by jevenski in Novik/ruTorrent#2841
* Improve move column function by jevenski in Novik/ruTorrent#2844
* Improve s-table header row hover event handler by jevenski in Novik/ruTorrent#2851
* Drag and drop s-table header using DnD class by jevenski in Novik/ruTorrent#2855
* Merge stable head and body into one table by jevenski in Novik/ruTorrent#2860
* Add danger color to #randomize_hash by anthonyryan1 in Novik/ruTorrent#2875
* Fix various bugs for v5.2 by jevenski in Novik/ruTorrent#2863
* Let rutracker_check ignore torrents with certain labels by drrako in Novik/ruTorrent#2866
* Add "::1" to localhost address list. by rozhuk-im in Novik/ruTorrent#2881
* Improve $tempDirectory handling: by rozhuk-im in Novik/ruTorrent#2880
* Diskspace option to display free bytes (new pull request against develop branch) by TheGoblinHero in Novik/ruTorrent#2890
* Add a permission to hide all torrent options by fffe in Novik/ruTorrent#2868
* Mark ruTorrent v5.2 as Stable by stickz in Novik/ruTorrent#2891

## New Contributors
* pekman made their first contribution in Novik/ruTorrent#2806
* rozhuk-im made their first contribution in Novik/ruTorrent#2881

**Full Changelog**: Novik/ruTorrent@v5.1.7...v5.2.0
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.

2 participants