Skip to content

Conversation

jevenski
Copy link
Contributor

@jevenski jevenski commented Dec 1, 2024

This patch does modifications to the context menu. After this patch is merged, the search menu on the top navbar can be more easily replaced with the bootstrap dropdown menu implementation, just like the plugins dropdown menu.

Commit Message

  • Split menu-item selection and sub-menu expansion icons from the background image, and use pseudo-elements for them.
  • Use CSS variables on menu background and font color. This simplifies theme adaption process and reduces code duplication.
  • Use inline base64 string for menu selection and expansion icons, replacing previous image urls. This saves a few requests for images and enhances performance by reducing load time.

- Split menu-item selection and sub-menu expansion icons from the
  background image, and use pseudo-elements for them.
- Use CSS variables on menu background and font color. This simplifies
  theme adaption process and reduces code duplication.
- Use inline base64 string for menu selection and expansion icons,
  replacing previous image urls. This saves a few requests for images
  and enhances performance by reducing load time.
Copy link
Collaborator

@stickz stickz left a comment

Choose a reason for hiding this comment

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

Remove Details... from CMenu in this pull request. This is useless because it simply links to the General tab bellow. The user can display this tab in 1 click instead of 2 clicks.

@koblack
Copy link
Contributor

koblack commented Dec 1, 2024

  1. The separation line is not centered
    image

  2. Too much space between menu and submenu
    image
    image
    image

jevenski and others added 2 commits December 2, 2024 14:44
- Remove "Details..." option from torrent context menu.
- Fix submenu position.
- Adjust context menu display styles.
@stickz
Copy link
Collaborator

stickz commented Dec 2, 2024

@jevenski Looks good, thanks! Merged. I added 0.25rem top and bottom padding to the CMenu. The text was too close to the edge when testing. I didn't add any left and right padding, as there is already padding for that.

@stickz stickz merged commit f6aa287 into Novik:develop Dec 2, 2024
@jevenski jevenski deleted the ctx-menu branch December 2, 2024 15:08
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.

3 participants