Skip to content

Add "Delete" shortcut, mod list download and steam path#1465

Draft
iMuhammadMustafa wants to merge 3 commits intoRimSort:mainfrom
iMuhammadMustafa:AddDeleteShortcutAndModListDownload
Draft

Add "Delete" shortcut, mod list download and steam path#1465
iMuhammadMustafa wants to merge 3 commits intoRimSort:mainfrom
iMuhammadMustafa:AddDeleteShortcutAndModListDownload

Conversation

@iMuhammadMustafa
Copy link
Copy Markdown

Hello

These are some things that I really needed to exist. I'm not very experienced with python though, so I apologize in advance.

  1. Add the ability to press "Delete" Key on the keyboard to invoke a delete command similar to selecting "Delete mod completely" from the context menu.
image
  1. Add the ability to view the steam url and click directly from the mod panel instead of selecting "Open URL in browser" from the contextt menu, and similar to the path.
image
  1. Add the ability to add a comma-seperated list of mod ids to the steam browser to download.
    (I needed this because I already had a list from long ago that I wanted to add and sometimes I'm browsing mods from a different laptop and create a list and want to automatically download all of them instead of browsing one by one.)
image image

@LionelColaso
Copy link
Copy Markdown
Member

3. Add the ability to add a comma-seperated list of mod ids to the steam browser to download.
   (I needed this because I already had a list from long ago that I wanted to add and sometimes I'm browsing mods from a different laptop and create a list and want to automatically download all of them instead of browsing one by one.)

Try searching for "steamcmd_script.txt" and see how it is used in case you have more ideas

image

logger.debug("Finished Browser Window initialization")

def _show_add_mods_by_id_dialog(self):
from PySide6.QtWidgets import QDialog, QVBoxLayout, QTextEdit, QDialogButtonBox, QLabel
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Move imports to top of the file to avoid duplicate imports

@LionelColaso
Copy link
Copy Markdown
Member

Should be an easy fix based on the error below also read incase https://rimsort.github.io/RimSort/development-guide/contributor-guidelines#misc-coding-style
image

if self.path.startswith("http://") or self.path.startswith("https://"):
import webbrowser
try:
webbrowser.open(self.path)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Use platform_specific_open instead of webbrowser.open works with links as well

@LionelColaso LionelColaso added feature/improvement 🆕 Use this label to mark new feature or improvement requests accessibility 👀 ℹ️🤓 user-friendly optimization, like notifications, warnings and action alerts labels Nov 5, 2025
@LionelColaso
Copy link
Copy Markdown
Member

You should also check these out if possible
#1365
#1243
#1014

@iMuhammadMustafa iMuhammadMustafa marked this pull request as draft November 6, 2025 01:27
@iMuhammadMustafa
Copy link
Copy Markdown
Author

Hi @LionelColaso Thank you so much for reviewing my PR. I moved the PR to draft while I review your comments and resolve it. Thank you

@LionelColaso
Copy link
Copy Markdown
Member

LionelColaso commented Nov 6, 2025

Hi @LionelColaso Thank you so much for reviewing my PR. I moved the PR to draft while I review your comments and resolve it. Thank you

You should probably create individual pull requests for each feature rather than implementing all at once

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

accessibility 👀 ℹ️🤓 user-friendly optimization, like notifications, warnings and action alerts feature/improvement 🆕 Use this label to mark new feature or improvement requests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants