torrust / torrust-index-gui

This repository serves as the frontend for the Torrust Index project.
https://torrust.com
Other
32 stars 16 forks source link

Update root Torrust logo #637

Open mario-nt opened 1 week ago

mario-nt commented 1 week ago

I think the Torrust logo should be updated to the new version:

image

@josecelano @cgbosse @da2ce7 @grmbyrn

josecelano commented 1 week ago

Hi @mario-nt what is shown is actually not the logo; it's a text you can set in the Index configuration:

[website]
name = "Torrust"

It would be nice to let users decide if they want a text or an image and let them change it.

Probably, the reason it has not been implemented yet it's because we don't have a proper media manager in the Index. There is no way to upload files. For the time being, the only way to update the theme would be to fork this repo and override the style.

We should think about the MVP for layout customizations. I think the minimum features would be:

But the first question is: Do we want to give the Index API the responsibility of storing UI settings? In our past discussions, we envision the Index as a database for classified torrents, not as the backend for this front-end app. But then the question is, where can we put that information?

Relates to:

cc @da2ce7

mario-nt commented 1 week ago

@josecelano Hi, I would store all frontend related data in the GUI (logos, icons, themes) to keep them in a separate layer and leave the index for other info regarding torrent files. Basically having the GUI communicating with the index's API only to get info about the torrent files.

That is what comes to my mind right now anyway.

I would like to give it a second thought and really consider the best approach to the responsibilities of each app (index and GUI).