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(linux): set environment variable for rendering of downloads dialog 🍒 🏠 #12617

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

ermshiperete
Copy link
Contributor

This works around a problem in webkit2gtk where the download dialog sometimes only renders a blank screen. Setting the environment variable seems to help, not only in the cases that the issue describes (with a NVidia graphics card), but also in my testing in a VM.

Fixes: #12587
Cherry-pick-of: #12616

User Testing

TEST_DOWNLOAD_PAGE: Open Keyman Config and click on the Download Keyboard button. The Download Keyman Keyboards page should show up.

This works around a problem in webkit2gtk where the download dialog
sometimes only renders a blank screen. Setting the environment variable
seems to help, not only in the cases that the issue describes (with a
NVidia graphics card), but also in my testing in a VM.

Fixes: #12587
Cherry-pick-of: #12616
@keymanapp-test-bot keymanapp-test-bot bot added has-user-test user-test-required User tests have not been completed labels Nov 4, 2024
@github-actions github-actions bot added linux/ linux/config/ and removed user-test-required User tests have not been completed labels Nov 4, 2024
@keymanapp-test-bot
Copy link

keymanapp-test-bot bot commented Nov 4, 2024

@keymanapp-test-bot keymanapp-test-bot bot changed the title fix(linux): set environment variable for rendering of downloads dialog fix(linux): set environment variable for rendering of downloads dialog 🏠 Nov 4, 2024
@keymanapp-test-bot keymanapp-test-bot bot added this to the A18S14 milestone Nov 4, 2024
@ermshiperete ermshiperete changed the title fix(linux): set environment variable for rendering of downloads dialog 🏠 fix(linux): set environment variable for rendering of downloads dialog 🍒 🏠 Nov 4, 2024
@keymanapp-test-bot keymanapp-test-bot bot added the user-test-required User tests have not been completed label Nov 4, 2024
@dinakaranr
Copy link

Test Results

I tested this issue with the attached "package version 17.0.332-1~PR-12617-3958.1+noble1" build on the Noble & Focal OS environment. I'm sharing my observation here.

  • TEST_DOWNLOAD_PAGE (Passed):
  1. Installed the Keyman keyboard package version 17.0.332-1~PR-12617-3958.1+noble1
  2. Open the configuration dialog by running the "km-config" command in the terminal.
  3. "Download Keyman keyboards" dialog by clicking the "Download keyboard" button.
  4. Installed the SIL IPA keyboard.(search a keyboard in the search box)
  5. Verified that the installed keyboard appears on the configuration dialog.
    It works great for me. Thank you.

@keymanapp-test-bot keymanapp-test-bot bot removed the user-test-required User tests have not been completed label Nov 5, 2024
@mcdurdin mcdurdin merged commit 34f55bb into stable-17.0 Nov 6, 2024
9 checks passed
@mcdurdin mcdurdin deleted the fix/linux/cherry-pick/12587_webkit branch November 6, 2024 02:07
@keyman-server
Copy link
Collaborator

Changes in this pull request will be available for download in Keyman version 17.0.332

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants