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

open map in landscape orientation and constrain default webxdc bounds by screen workarea #4683

Merged
merged 3 commits into from
Feb 20, 2025

Conversation

Simon-Laux
Copy link
Member

  • open map in landscape orientation and with a bigger window
  • constrain size of webxdc window by available screen-workarea-space

closes #3847

… by screen workarea

- open map in landscape orientation and with a bigger window
- constrain size of webxdc window by available screen-workarea-space

closes #3847
@ralphtheninja
Copy link
Member

ralphtheninja commented Feb 20, 2025

I just tested this locally but the map looks like a normal webxdc

Opened by

image

image

I'm guessing this happens because I have opened the map before and it has saved it's bounds already. So this fix only has effect for people opening the map for the first time, right?

@ralphtheninja
Copy link
Member

@Simon-Laux I can make a PR into this branch instead. It's easier to show what I mean that way. See if you like it or not.

@Simon-Laux
Copy link
Member Author

Simon-Laux commented Feb 20, 2025

I'm guessing this happens because I have opened the map before and it has saved it's bounds already. So this fix only has effect for people opening the map for the first time, right?

yes it only affects new/fresh accounts

BTW: I don't like the whole way of reusing webxdc for the map and abusing the update mechanism, which only allows one instance of the map currently. IMO we need to think about a good way to do deltachat extensions, as their own thing, separate from webxdc. Like expose a dedicated jsonrpc api with autogenerated bindings that you can easily develop extensions and don't need to invent a new RPC mechanism over sending status updates that doesn't even do error handling and has no typescript types. Also the map xdc itself still lacks features that the old map box map had. But enough ranting, for now it is what it is and we will need some time in the future where we focus on it to make it really nice.

ralphtheninja and others added 2 commits February 21, 2025 00:10
* Pass in default size instead of booleans

* Put the comment back on why map is using a different size

* Always compute the default size at the last minute + document adjust function

* Adjust all bounds

* Adjust size separately

* Remove default
@Simon-Laux Simon-Laux merged commit 57cfd8f into main Feb 20, 2025
8 of 9 checks passed
@Simon-Laux Simon-Laux deleted the simon/i3847 branch February 20, 2025 23:19
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

Successfully merging this pull request may close these issues.

Change map default dimensions to be bigger and landscape instead of portrait
2 participants