Skip to content

Material design theme patched to use svg icons #2873

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

silasmariusz
Copy link

Please apply above patch to allow customers properly use RuTOrrent responsive interface on mobile phones. In future please also rewrite how icons are rendered on left side bar. Its impossible to replace background sprite with own .svg files.

(This is Material Design improvement/patch only)

Once Ill be albe to replae icons on left sidebar I could prepare all themes patch and on internal css styles.

@stickz
Copy link
Collaborator

stickz commented Mar 22, 2025

Thank you for the pull request. We will review this and incorporate it into ruTorrent.

Could you please submit to the develop instead of master branch? We need to test before we ship it.

@silasmariusz
Copy link
Author

Hi @stickz
Within this update Material Design (Dark) seems to be looking very well on mobile/tablet.
rtorrent_icons
... except the left bar ofcourse.
I need to ask you for an advise or maybe help to provide guide/method how we can smoothly replace old background sprites with IMG tags or DIV blocks with CSS style (background-image url(.svg)

After that I cpould move on and bascically maybe try to prepare very similar MaterialDesign Light version with color scheme (I think) exactly the same one like here on github.

But no rush, just please considder this someday.

Thank you

@stickz
Copy link
Collaborator

stickz commented Mar 23, 2025

Hi @silasmariusz, can you change this pull request to the develop branch?

I agree with changing all images to SVG. My advice is to use multiple pull requests, to split the task into smaller pieces.

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