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

Font size issue in v0.14.0 #4717

Closed
vsemozhetbyt opened this issue Apr 14, 2016 · 18 comments
Closed

Font size issue in v0.14.0 #4717

vsemozhetbyt opened this issue Apr 14, 2016 · 18 comments
Assignees
@rogerwang
Milestone
0.14.x

Comments

@vsemozhetbyt
Copy link
Contributor

  • NW.js v0.14.0
  • Windows 7 x64

I have a simple test app:

{
  "name": "nw-test",
  "main": "index.html",
  "window": {
    "title": "NW Test"
  }
}
<!DOCTYPE html>
<html>
  <head>
    <meta charset='UTF-8'>
    <title>NW Test</title>
  </head>
  <body>
    <pre><script>
      document.write(JSON.stringify(process.versions, null, '  '));
    </script></pre>
  </body>
</html>

Strangely if I open it in the nwjs-sdk, the font size is normal, but in the nwjs-normal the font size is extremely small. Deleting app folder in the user appdata does not fix the issue. The usual Chrome Ctrl++, Ctrl+-, Ctrl+0 don't change the font size in both nw flavors.
@rogerwang rogerwang added this to the 0.14.x milestone Apr 14, 2016
@rogerwang rogerwang self-assigned this Apr 14, 2016
@rogerwang
Copy link
Member

I couldn't reproduce this in my system. Could you please follow https://www.chromium.org/for-testers/enable-logging and post the log files for both versions?

@vsemozhetbyt
Copy link
Contributor Author

SDK screenshot and debug log:

sdk

chrome_debug.sdk.txt

Normal screenshot and debug log:

normal

chrome_debug.normal.txt

@vsemozhetbyt
Copy link
Contributor Author

Command lines (check if I get it right):

...\nwjs-sdk\nw.exe --enable-logging --v=1 ...\test\

...\nwjs\nw.exe --enable-logging --v=1 ...\test\

@rogerwang
Copy link
Member

Does it happen in 0.13.4?

@vsemozhetbyt
Copy link
Contributor Author

@rogerwang
0.13.4 is OK.
0.14.1 has the same issue.

@rogerwang
Copy link
Member

Did you use 'package.json' or 'manifest.json' as the filename for the manifest? And do you have all the files in 'nwjs' directory as we distributed in our official zip file?

@vsemozhetbyt
Copy link
Contributor Author

vsemozhetbyt commented Apr 20, 2016 •
edited
Loading

package.json

I don't remove or add any files from/to distributed folder after unzipping.

@wJsJwr
Copy link

wJsJwr commented Apr 21, 2016

Same problem here on Windows 10 x64 with 0.14.x.

@hooker99
Copy link

hooker99 commented Apr 21, 2016 •
edited
Loading

Me too (small fonts): Windows 10 x64 with 0.14.1 (only normal version, sdk is fine)

@vsemozhetbyt
Copy link
Contributor Author

Seems to be fixed in the 0.14.2.

@rogerwang
Copy link
Member

Yeah. we fixed a related issue #4746 in 0.14.2. glad to see it helps this issue.

@PriteshGajjar008
Copy link

@rogerwang I am using 0.14.7 and Ctrl++/Ctrl-- not working for my application. Please confirm.

@rogerwang
Copy link
Member

cc @Christywl

@Christywl
Copy link
Contributor

Ctrl++ and Ctrl-- don't work on nwjs-sdk-v0.21. But I think it should be another issue, not same with this bug.

@PriteshGajjar008
Copy link

So you want me to raise new bug?

@PriteshGajjar008
Copy link

I am using 0.14.7 and it doesn't work on that as well.

@rogerwang
Copy link
Member

@PriteshGajjar008 It shouldn't be a default behavior. you can set NWWindow.zoomLevel and bind it to the hotkeys.

@PriteshGajjar008
Copy link

Thanks, got it. I could register this shortcut.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rogerwang rogerwang

Labels
None yet
Projects
None yet
Milestone
0.14.x
Development

No branches or pull requests
6 participants
@rogerwang @wJsJwr @vsemozhetbyt @hooker99 @Christywl @PriteshGajjar008