Skip to content

Commit

Permalink
Merge branch 'master' of https://github.com/neroist/happyx-native
Browse files Browse the repository at this point in the history
  • Loading branch information
neroist committed Jun 10, 2024
2 parents 2674dbc + c1a24db commit 3d4c9f7
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 5 deletions.
4 changes: 4 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -108,6 +108,10 @@ for you. This is due to a limitation within Happyx Native, that will be resolved
future. Currently, you may manually link in your desired window icon, like how is done in
<https://github.com/neroist/webview/tree/main/examples/example_application/windows>.

In addition, Webview does not currently support window positioning
(see <https://github.com/webview/webview/issues/642>), so the `x` and `y` arguments passed to
`nativeApp` will be ignored.

## Browsers

If you want choose other browser instead of default - use these flags:
Expand Down
5 changes: 0 additions & 5 deletions src/happyx_native/app/app.nim
Original file line number Diff line number Diff line change
Expand Up @@ -68,9 +68,6 @@ macro callback*(body: untyped) =
var params: seq[NimNode] = @[]
for param in statement.params:
params.add(param)
dumpTree:
{.gcsafe.}:
echo 1
var prc = newProc(
statement.name,
params,
Expand Down Expand Up @@ -117,8 +114,6 @@ macro callback*(body: untyped) =
nnkTemplateDef
)
)
echo result.toStrLit


macro callJsAsync*(funcName: string, params: varargs[untyped]) =
quote do:
Expand Down

0 comments on commit 3d4c9f7

Please sign in to comment.