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

Failing to get the right windows, documents, bodies in some MessageOverlay methods #106

Closed
eyalroz opened this issue Aug 14, 2023 · 0 comments
Assignees
Labels
bug Something isn't working resolved-on-development

Comments

@eyalroz
Copy link
Owner

eyalroz commented Aug 14, 2023

The code in methods such as gatherParameters(), forceDirection() and cycleDirectionSettings(), we use the messagePane, which we locate using TB-102-compatible code - that can't be used in TB 115. Instead, let's use the new "about Message" mechanism; it can be used like so:

let aboutMessage =
  document.getElementById("tabmail")?.currentAboutMessage ||
  document.getElementById("messageBrowser").contentWindow;
@eyalroz eyalroz added the bug Something isn't working label Aug 14, 2023
@eyalroz eyalroz self-assigned this Aug 14, 2023
eyalroz added a commit that referenced this issue Aug 14, 2023
eyalroz added a commit that referenced this issue Aug 15, 2023
eyalroz added a commit that referenced this issue Aug 16, 2023
…ocuments and bodies

Note: Some of the code for doing this leaves much to be desired:

* It may be possible to avoid passing the window to a wrapper to the `onLoad()` function, and down into various getters
* It may be possible to detect when the global window will not have a `tabmail` or `messageBrowser` elements; and perhaps one can conclude apriori that in this case it has a `messagePaneBrowser`
* We may be overly protective with the `?.` operators
@eyalroz eyalroz closed this as completed in 2050920 Sep 1, 2023
eyalroz added a commit that referenced this issue Sep 1, 2023
…ocuments and bodies

Note: Some of the code for doing this leaves much to be desired:

* It may be possible to avoid passing the window to a wrapper to the `onLoad()` function, and down into various getters
* It may be possible to detect when the global window will not have a `tabmail` or `messageBrowser` elements; and perhaps one can conclude apriori that in this case it has a `messagePaneBrowser`
* We may be overly protective with the `?.` operators
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working resolved-on-development
Projects
None yet
Development

No branches or pull requests

1 participant