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

White border on main window with frame:true (was Dark grey on previous versions) #7073

Closed
vankasteelj opened this issue Jun 10, 2019 · 5 comments

Comments

@vankasteelj
Copy link

vankasteelj commented Jun 10, 2019

Hi,

NWJS Version : 0.38.4-sdk
Operating System : Windows 10

Expected behavior

The "frame", the outline of the window was invisible, it's now a 1 or 2px white line. On Chrome, it's a black line.

Actual behavior

The "outline" of the app is white now, for some reason. The devtool window outline is black however.

How to reproduce

open a nwjs window with "frame: true"

============

Here are two screenshots: 1 of the main window on a colored background (to show the white), 1 of the devtool window on a white background (to show the black).

2019-06-10_09-52-29
Brackets_2019-06-10_09-52-59

@TheJaredWilcurt
Copy link
Member

That 1px (or 2px on HDPI screens) border comes from Windows Theme color.

Windows Theme settings and an NW.js app

The border will inherit the theme color. When the app is not in focused it will be black or a very dark version of the color (orange -> dark brown in the above example). You can make a frameless app if you want complete control of all the pixels, but will need to create your own buttons for minimize/maximize/restore/close.

Example:

screenshot of an app

@vankasteelj
Copy link
Author

Right, but that doesn't explain why there's a black border on Chrome (itself & devtool) and nwjs' devtool, while it's white on nwjs itself.

@TheJaredWilcurt
Copy link
Member

Because those windows are not opting in to the OS's color theme. The devtools use a light gray title bar. You need to acknowledge that the frame surrounding your app will be different in each OS. You can either let your app live inside a standard framed window that the OS gives you, or remove the frame to have complete control over the UI in all OS's.

@vankasteelj
Copy link
Author

Let me phrase this another way:

  • 0.32.4: dark grey border
  • 0.36.4: dark grey border
  • 0.37.4: white border
  • 0.38.4: white border

I'm asking for the return of the dark grey border.

Examples with 0.32.4 vs 0.37.4:
2019-06-17_08-07-26
2019-06-17_08-08-06

@vankasteelj vankasteelj changed the title White frame on main window, black on devtools White border on main window with frame:true (was Dark grey on previous versions) Jun 17, 2019
vankasteelj added a commit to vankasteelj/frak that referenced this issue Jun 17, 2019
- newer versions use a white border
- see nwjs/nw.js#7073
@vankasteelj
Copy link
Author

as of 0.44.3, the dark grey border is back!

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