WebUI: Allow to move state icon to name column in torrents table #22118
Conversation
|
This is not a good solution. Some users on the contrary request separate columns for the GUI #18193 I would suggest making a full column for WebUI called “Status Icon” or just “Icon” instead of merging. |
I see, thanks for letting me know. It doesn't hurt to keep the old state icon column so I'll modify the PR and bring it back. |
skomerko
force-pushed
the
webui-state-icon-column
branch
from
ad6fafb to
08272b6
Compare
skomerko
changed the title
|
PR has been updated. Notable changes:
edit: PR title probably should be changed(?), not sure to what though. |
|
How about the name “Icon”? #22131 (comment)
|
| @@ -612,31 +628,21 @@ window.qBittorrent.DynamicTable ??= (() => { | |||
| return -1; | |||
| }, | |||
|
|
|||
| updateColumn: function(columnName) { | |||
| updateColumn: function(columnName, updateCellData = false) { | |||
There was a problem hiding this comment.
@Chocobo1, changes made around this method are not strictly necessery - name column could be updated from within onVisibilityChange callback but I thought it's better to improve updateColumn a bit instead (though things are definitely not perfect but I didn't want to change too much).
| this.columns["state_icon"].updateTd = function(td, row) { | ||
| let state = this.getRowValue(row); | ||
| let img_path; | ||
| const getStateIconClasses = (state) => { |
There was a problem hiding this comment.
Shouldn't it be singular?
| const getStateIconClasses = (state) => { | |
| const getStateIconClass = (state) => { |
There was a problem hiding this comment.
It returns a string containing two classes so I think its fine?? Should I still rename it?
There was a problem hiding this comment.
It returns a string containing two classes so I think its fine??
OK, never mind.
Also, I wonder is it wise to merge stateIcon into stateClass? It now needs 2 classes while only 1 should be sufficient. I don't insist it be done now, i.e. it could be done later.
skomerko
force-pushed
the
webui-state-icon-column
branch
from
08272b6 to
a3da699
Compare
|
There is a naming inconsistency when using “State Icon”. |
skomerko
force-pushed
the
webui-state-icon-column
branch
from
a3da699 to
56fc833
Compare
Sure, changed it to 'Status Icon'. |
See #22118 (comment) for updated information.