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

fix: don't set delegate for QLPreviewPanel #37578

Conversation

trop[bot]
Copy link
Contributor

@trop trop bot commented Mar 14, 2023

Backport of #37530

See that PR for details.

Notes: Fixed issue with BrowserWindow not updating after call to previewFile.

@trop trop bot mentioned this pull request Mar 14, 2023
5 tasks
@trop trop bot added 24-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels Mar 14, 2023
Copy link
Contributor

@jkleinsc jkleinsc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Newly added test fails.

@codebytere codebytere force-pushed the trop/24-x-y-bp-fix-don-t-set-delegate-for-qlpreviewpanel--1678801313961 branch from ab56cd6 to 089dfed Compare March 16, 2023 08:54
Co-authored-by: Jeremy Spiegel <jeremy.spiegel@frontapp.com>
@codebytere codebytere force-pushed the trop/24-x-y-bp-fix-don-t-set-delegate-for-qlpreviewpanel--1678801313961 branch from 089dfed to fae017f Compare March 16, 2023 10:31
@jkleinsc jkleinsc merged commit 080fb8e into 24-x-y Mar 21, 2023
@jkleinsc jkleinsc deleted the trop/24-x-y-bp-fix-don-t-set-delegate-for-qlpreviewpanel--1678801313961 branch March 21, 2023 00:17
@release-clerk
Copy link

release-clerk bot commented Mar 21, 2023

Release Notes Persisted

Fixed issue with BrowserWindow not updating after call to previewFile.

vikunja-bot pushed a commit to go-vikunja/desktop that referenced this pull request Mar 23, 2023
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [electron](https://github.com/electron/electron) | devDependencies | minor | [`23.1.4` -> `23.2.0`](https://renovatebot.com/diffs/npm/electron/23.1.4/23.2.0) |

---

### Release Notes

<details>
<summary>electron/electron</summary>

### [`v23.2.0`](https://github.com/electron/electron/releases/tag/v23.2.0): electron v23.2.0

[Compare Source](electron/electron@v23.1.4...v23.2.0)

### Release Notes for v23.2.0

#### Features

-   Added the `enableLocalEcho` flag to the session handler `ses.setDisplayMediaRequestHandler()` callback for allowing remote audio input to be echoed in the local output stream when `audio` is a `WebFrameMain`. [#&#8203;37529](electron/electron#37529) <span style="font-size:small;">(Also in [24](https://github.com/electron/electron/pull/37528))</span>

#### Fixes

-   Fixed an issue where `BroadcastChannel` did not work correctly when `contextIsolation: false`. [#&#8203;37444](electron/electron#37444) <span style="font-size:small;">(Also in [24](https://github.com/electron/electron/pull/37443))</span>
-   Fixed an issue with potential dock icon duplication on macOS. [#&#8203;37624](electron/electron#37624) <span style="font-size:small;">(Also in [22](electron/electron#37623), [24](https://github.com/electron/electron/pull/37625))</span>
-   Fixed issue with BrowserWindow not updating after call to previewFile. [#&#8203;37577](electron/electron#37577) <span style="font-size:small;">(Also in [22](electron/electron#37576), [24](https://github.com/electron/electron/pull/37578))</span>
-   Improved error messages on `session.cookies.set` failure. [#&#8203;37596](electron/electron#37596) <span style="font-size:small;">(Also in [22](electron/electron#37595), [24](https://github.com/electron/electron/pull/37597))</span>

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNC4xNTcuMCIsInVwZGF0ZWRJblZlciI6IjM0LjE1Ny4wIn0=-->

Reviewed-on: https://kolaente.dev/vikunja/desktop/pulls/143
Co-authored-by: renovate <renovatebot@kolaente.de>
Co-committed-by: renovate <renovatebot@kolaente.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
24-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants