openscopeproject / TrguiNG

Remote GUI for Transmission torrent daemon
GNU Affero General Public License v3.0
372 stars 47 forks source link

Add ability to resize / auto-size columns #31

Closed wikiped closed 1 year ago

wikiped commented 1 year ago

Currently (v0.5.0-dirty) all columns have the same width and there are no column separators to drag / resize columns.

But otherwise good alternative gui. Thank you.

qu1ck commented 1 year ago

Does the resizer not appear when you hover over the column header?

image

Post your frontend info, it can be found in application info below the version.

melyux commented 1 year ago

Resizing works for me, but auto-resizing when double clicking the resizer would be a good addition

qu1ck commented 1 year ago

I looked into auto sizing, it's not easy to implement and given that table contents updates all the time it would be of limited use. Maybe I'll add it at some point but don't expect it soon.

Pentaphon commented 1 year ago

I don't think auto-sizing is very useful here. The existing column sizing is already powerful enough. I prefer tweaking the column sizes myself because auto-sizing usually leads to results that don't suit me.

wikiped commented 1 year ago

Does the resizer not appear when you hover over the column header?

For some reason resizer does not appear. So I thought it was not "supposed" to be there for now, hence the issue.

This is what it looks like for me:

Dark skin:

image

White skin:

image

And nothing changes when I hover over the header.

Perhaps a "front-end" issue? This is what is shown in "Version Info" window: Edge 114.0.1823.67 (Blink 114.0.0.0) on Windows 10

qu1ck commented 1 year ago

Strange, I have exact same edge version on win 10 and I don't have this issue.

Is there anything unusual about your setup, like non standard mouse or maybe you are using remote desktop software?

wikiped commented 1 year ago

I can't say there is something unusual - no other program has shown similar symptoms...

I stumbled upon Inspect menu which helped to get to html/css properties of header:

image

So technically resizer is there, but it's width is 0, therefore I can't see/drag it.

wikiped commented 1 year ago

I can see that css properties have been read:

image

But I also noticed two warning messages in the log:

DevTools failed to load source map: Could not load content for https://tauri.localhost/main.bundle.css.map: Connection error: net::ERR_CONNECTION_REFUSED
DevTools failed to load source map: Could not load content for https://tauri.localhost/737.bundle.css.map: Connection error: net::ERR_CONNECTION_REFUSED

Perhaps the latter is only relevant for DeTools?

I checked my firewall settings for msedgewebview2.exe and trguing.exe and I don't see any reason why the connection would be refused.

qu1ck commented 1 year ago

These rules are supposed to apply and make it narrow and mostly invisible normally but thicker and more visible when hovered over header

https://github.com/openscopeproject/TrguiNG/blob/master/src/css/torrenttable.css#L93-L103

Looks like some styles load but the @media(hover) don't, interesting.

Error loading css maps is irrelevant.

qu1ck commented 1 year ago

Are you on a laptop with a touchscreen perhaps? That's the only reason I can think of that those wouldn't work. I can rewrite the rules without @media query and see if it helps.

qu1ck commented 1 year ago

Here, try this build. TrguiNG_0.6.0_x64-setup.exe.zip

wikiped commented 1 year ago

Are you on a laptop with a touchscreen perhaps?

Yes, the laptop is with the touchscreen, but it is not "in use" - the actual display is HDMI connected (perhaps this is unusual...)

Tried new build - and yes, the resizer is now visible and usable. Thank you.