Skip to content
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

[nightly-202401180913] can't move window using title bar #1406

Closed
ilike2burnthing opened this issue Jan 19, 2024 · 10 comments
Closed

[nightly-202401180913] can't move window using title bar #1406

ilike2burnthing opened this issue Jan 19, 2024 · 10 comments

Comments

@ilike2burnthing
Copy link
Contributor

What build of DuckieTV are you using (Standlone / Chrome Extension (New Tab / Browser Action))
Standalone

What version of DuckieTV are you using (Stable 1.1.x / Nightly yyyymmddHHMM)
Nightly 202401180913

What is your Operating System (Windows, MacOSx, Linux)
Windows 10

Describe the problem you are having and steps to reproduce if available

  • install and run nightly-202311011754
  • move mouse to DuckieTV's title bar
  • click, hold, and drag to move window

  • install and run nightly-202401180913
  • move mouse to DuckieTV's title bar - mouse icon changes to move icon (4 arrows)
  • click, hold, and attempt to drag to move window, but fail

Can maximise, restore, and minimise; can use Windows key plus arrow keys to snap window; and confirmed in Windows Sandbox that a clean install behaves the same.

Attach any DuckieTV statistics or Developer Console logs if available
...

@garfield69
Copy link
Collaborator

There is no title bar as the window is frameless.
Those min/max/close buttons on the top right are not the win10 frame buttons but glyphicon hardcoded in the tabs.html
I forget the reason we dont use regular windows, I expect it was to gain more real-estate for presentation.

@ilike2burnthing
Copy link
Contributor Author

Ah kk, well the top of the window then. It was previously clickable and dragable, it no longer is as of yesterday's release.

@garfield69
Copy link
Collaborator

garfield69 commented Jan 19, 2024

ok. i did switch NWJS to 0.83.0 (chromium 120) from 0.81.1 (chromium 118) in the latest build so it could be a fallout or regression from that.
I'll see if there is any tickets open for it on nwjs but lately there has been little ticket solving there, other than the regular bumping of version to import the latest nodejs and chromium updates.

@garfield69
Copy link
Collaborator

possible that nwjs/nw.js#8153 is relevant.

@ilike2burnthing
Copy link
Contributor Author

Looks it.

@garfield69
Copy link
Collaborator

I'll test reverting the nwjs back to 0.81 and check that the installer does not cause an issue on my sandbox for ubuntu and win10.
Sometimes upgrading chromium can cause backward compatibility error messages, so touch wood that we don't get one of those situations where we have to wait for the nwjs ticket to be solved with the next nwjs release.

@garfield69
Copy link
Collaborator

@ilike2burnthing
Copy link
Contributor Author

Ta, working well.

@garfield69 garfield69 reopened this Feb 13, 2024
@garfield69
Copy link
Collaborator

nwjs/nw.js#8153 (comment)

Awaiting the release of NWJS 0.85.0 with chromium 122 for the window drag fix.

@garfield69
Copy link
Collaborator

should be resolved with https://github.com/DuckieTV/Nightlies/releases/tag/Nightly-202404110853 which uses NWJS 0.86 with Chromium 123

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants