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

Add map as internal webxdc widget #3810

Closed
wants to merge 34 commits into from
Closed

Add map as internal webxdc widget #3810

wants to merge 34 commits into from

Conversation

nicodh
Copy link
Contributor

@nicodh nicodh commented May 7, 2024

Depends on deltachat/deltachat-core-rust#5541 !!

Depends on non public package @deltachat/stdio-rpc-server !!

Added in chat view and as global map (new)

MR to stdio branch, since the new methods
setWebxdcIntegration
initWebxdcIntegration

do not exist in deltachat-node

tbd: naming of Map Widgets
The title of the Map popup is relevant as it is possible to open multiple wirndows for multiple accounts
We have now:
Maps - Global map for all chats (Account 'AccountName')
Maps - Map for chat 'ChatName'

@nicodh nicodh requested a review from Simon-Laux May 7, 2024 11:28
@Simon-Laux
Copy link
Member

MR to stdio branch, since the new methods
setWebxdcIntegration
initWebxdcIntegration
do not exist in deltachat-node

you could also still have used locally compiled deltachat-node

@Simon-Laux Simon-Laux requested a review from r10s May 7, 2024 11:48
@Simon-Laux Simon-Laux added the new-core involves or requires an dc-node/core update label May 9, 2024
@Simon-Laux Simon-Laux force-pushed the stdio-experiment-with-jikstra branch from b01191b to 7d3654a Compare May 14, 2024 16:22
Simon-Laux and others added 25 commits May 14, 2024 18:27
Todo:
- [ ] account migration function in core
- [ ] port into own npm module that has all the prebuilds somehow
- [ ] ? fix open multiple instances
- [ ] code review from "experts" -> fix linereader/performance
I'm still thinking how to simplify that
app source will come in next commit
also reorder some imports in the files that I touched
…s pr in the end to make sure it does not end up in the main tree as it is quite a large file)
(mainly to allow the new syntax for importing json from esm modules in `@deltachat/stdio-rpc-server`, but also to)

don't mind the local paths too much they will be changed before this pr is merge ready
@Simon-Laux Simon-Laux force-pushed the stdio-experiment-with-jikstra branch from 7d3654a to da19267 Compare May 14, 2024 16:28
Simon-Laux and others added 5 commits May 15, 2024 19:30
Electron < 30.05 crashes on maximize

electron/electron#41839
@Simon-Laux Simon-Laux force-pushed the stdio-experiment-with-jikstra branch from 76b9a57 to a3f1a85 Compare May 17, 2024 10:17
Base automatically changed from stdio-experiment-with-jikstra to master May 17, 2024 18:06
@nicodh
Copy link
Contributor Author

nicodh commented May 20, 2024

Moved to #3836

@nicodh nicodh closed this May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-core involves or requires an dc-node/core update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants