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.

Sign up for GitHub

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

Jump to bottom

the transparent not work on v0.23.5! #5989

Closed
xingyuzhe opened this issue Jun 27, 2017 · 12 comments
Closed

the transparent not work on v0.23.5! #5989

xingyuzhe opened this issue Jun 27, 2017 · 12 comments
Labels
needinfo

Comments

@xingyuzhe
Copy link

xingyuzhe commented Jun 27, 2017 •
edited
Loading

the transparent not work at all event after the release of 0.23.5!
version: 0.23.5-sdk-ia32
os: win7

qq 20170627104220

The whole window change from tranparent to black when i week up my pc! i can produce this all the time.

to produce this bug, follow these steps:

  • minimize your nw window
  • lock your pc
  • week up your pc
  • restore your nw window
  • the window's bg color change to black!
@rogerwang
Copy link
Member

Did 0.23.4 work for you?

@rogerwang
Copy link
Member

CC @jtg-gg

@xingyuzhe
Copy link
Author

xingyuzhe commented Jun 27, 2017 •
edited
Loading

@rogerwang this bug appears since 0.22.3

@EdvaldoLima
Copy link

I have a similar problem, my case appears a menu inactive inside the window, which makes it impossible to work with totally transparent windows, to minimize the window it "disappears" but it is like flashing in the process of restoring the window.

Windows 10 x64 system.
Test in nwjs 0.23.4 and 0.23.5.

transparent

@jtg-gg
Copy link
Member

jtg-gg commented Jul 5, 2017

@xingyuzhe @EdvaldoLima may I have the sample app?
I can't reproduce it with mine

@EdvaldoLima
Copy link

EdvaldoLima commented Jul 5, 2017 •
edited
Loading

@jtg-gg

Project.
AppTest.zip

I made a short video so that you can better understand the application's implementation when executed.

issue #5989 see in HD

@Christywl
Copy link
Contributor

I can reproduce EdvaldoLima's issue with nwjs-sdk-v0.23.5.

  1. transparent doesn't work, it's same as window transparent not work on v0.23.2 #5973. Maybe the patch isn't merged in nw 0.23.5? Please help check.
"window": {
  "frame": false,
  "transparent": true
}
  1. EdvaldoLima's the transparent not work on v0.23.5! #5989 (comment)

to minimize the window it "disappears" but it is like flashing in the process of restoring the window.

The following issue only happens on Windows 10, doesn't happen on Windows 8.

And why does transparent work if add "resizable": false in package.json, maybe another issue?

  "window":{
    "frame": false,
    "transparent": true,
    "resizable": false
  },

@rogerwang @jtg-gg , please help check.

The bug reporter's issue may be different. @xingyuzhe , could you please provide a sample? And could you try other platforms to check if win7 32 specific issue? It works fine for me on Windows 7 x64 and Windows 8(I have no win7 32 PC).
<body style="background-color: transparent">

@jtg-gg
Copy link
Member

jtg-gg commented Jul 6, 2017

@EdvaldoLima have you tried to use these "--disable-gpu --force-cpu-draw" params as described in http://nwjs.readthedocs.io/en/latest/For%20Users/Advanced/Transparent%20Window/

@xingyuzhe
Copy link
Author

@jtg-gg --disable-gpu --force-cpu-draw works for me. And we use version sdk-ia32 for all x64 system.

@EdvaldoLima
Copy link

EdvaldoLima commented Jul 6, 2017 •
edited
Loading

@jtg-gg

The "--disable-gpu --force-cpu-draw" works perfectly, but with this flag I can reproduce another problem, the window becomes deficient to the event "focus" leaving practically the whole window with this problem.

Look --disable-gpu --force-cpu-draw

@jtg-gg jtg-gg mentioned this issue Jul 10, 2017
Closed
@jtg-gg
Copy link
Member

jtg-gg commented Jul 10, 2017

@EdvaldoLima I've just made a pull request nwjs/chromium.src#87
@rogerwang please accept my pull request, and also patch nw24

@rogerwang
Copy link
Member

This is fixed in git and will be available in the next nightly build.

@aprilandjan aprilandjan mentioned this issue Jul 24, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
needinfo
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@rogerwang @xingyuzhe @jtg-gg @EdvaldoLima @Christywl