Add "Delete" shortcut, mod list download and steam path#1465
Draft
iMuhammadMustafa wants to merge 3 commits intoRimSort:mainfrom
Draft
Add "Delete" shortcut, mod list download and steam path#1465iMuhammadMustafa wants to merge 3 commits intoRimSort:mainfrom
iMuhammadMustafa wants to merge 3 commits intoRimSort:mainfrom
Conversation
Member
LionelColaso
reviewed
Nov 5, 2025
| logger.debug("Finished Browser Window initialization") | ||
|
|
||
| def _show_add_mods_by_id_dialog(self): | ||
| from PySide6.QtWidgets import QDialog, QVBoxLayout, QTextEdit, QDialogButtonBox, QLabel |
Member
There was a problem hiding this comment.
Move imports to top of the file to avoid duplicate imports
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 |
LionelColaso
reviewed
Nov 5, 2025
| if self.path.startswith("http://") or self.path.startswith("https://"): | ||
| import webbrowser | ||
| try: | ||
| webbrowser.open(self.path) |
Member
There was a problem hiding this comment.
Use platform_specific_open instead of webbrowser.open works with links as well
Member
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 |
Member
You should probably create individual pull requests for each feature rather than implementing all at once |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Hello
These are some things that I really needed to exist. I'm not very experienced with python though, so I apologize in advance.
(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.)