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

feature: users can specify an app id for the wayland surface when running on the wayland platform #3614

Merged
merged 4 commits into from
Sep 26, 2024

Conversation

mattkae
Copy link
Contributor

@mattkae mattkae commented Sep 25, 2024

What's new?

  • Added the --wayland-surface-app-id=<APP_ID> option for the wayland platform that, when set, will call xdg_top_level::set_app_id with the value APP_ID
  • Added the --wayland-surface-title=<TITLE> option for the wayland platform that, when set, will call xdg_top_level::set_title with the value TITLE

@mattkae mattkae requested a review from Saviq September 25, 2024 15:22
@mattkae mattkae requested a review from a team as a code owner September 25, 2024 15:22
Copy link
Collaborator

@Saviq Saviq left a comment

Choose a reason for hiding this comment

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

Should this be --wayland-surface-app-id since that's the Wayland nomenclature?

Should we go for --wayland-surface-title as well?

@mattkae
Copy link
Contributor Author

mattkae commented Sep 25, 2024

Should this be --wayland-surface-app-id since that's the Wayland nomenclature?

Should we go for --wayland-surface-title as well?

We don't have a wayland-window-title currently, but yes to the wayland-surface-app-id!

@Saviq
Copy link
Collaborator

Saviq commented Sep 25, 2024

We don't have a wayland-window-title currently

That's what I mean - should we add it, in one go?

@mattkae
Copy link
Contributor Author

mattkae commented Sep 25, 2024

We don't have a wayland-window-title currently

That's what I mean - should we add it, in one go?

Sure!

@mattkae mattkae requested a review from Saviq September 26, 2024 12:52
Copy link
Collaborator

@Saviq Saviq left a comment

Choose a reason for hiding this comment

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

Legit!

@Saviq Saviq enabled auto-merge September 26, 2024 12:54
@Saviq Saviq added this pull request to the merge queue Sep 26, 2024
github-merge-queue bot pushed a commit that referenced this pull request Sep 26, 2024
…ning on the wayland platform (#3614)

## What's new?
- Added the `--wayland-surface-app-id=<APP_ID>` option for the wayland
platform that, when set, will call
[xdg_top_level::set_app_id](https://wayland.app/protocols/xdg-shell#xdg_toplevel:request:set_app_id)
with the value `APP_ID`
- Added the `--wayland-surface-title=<TITLE>` option for the wayland
platform that, when set, will call
[xdg_top_level::set_title](https://wayland.app/protocols/xdg-shell#xdg_toplevel:request:set_title)
with the value `TITLE`
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 26, 2024
@Saviq Saviq added this pull request to the merge queue Sep 26, 2024
Merged via the queue into main with commit bbb9278 Sep 26, 2024
37 checks passed
@Saviq Saviq deleted the feature/wayland_platform_app_id branch September 26, 2024 13:53
Saviq added a commit that referenced this pull request Sep 26, 2024
…ning on the wayland platform (#3614)

## What's new?
- Added the `--wayland-surface-app-id=<APP_ID>` option for the wayland
platform that, when set, will call
[xdg_top_level::set_app_id](https://wayland.app/protocols/xdg-shell#xdg_toplevel:request:set_app_id)
with the value `APP_ID`
- Added the `--wayland-surface-title=<TITLE>` option for the wayland
platform that, when set, will call
[xdg_top_level::set_title](https://wayland.app/protocols/xdg-shell#xdg_toplevel:request:set_title)
with the value `TITLE`
Saviq added a commit that referenced this pull request Sep 27, 2024
Enhancements:
. Allow to specify an app id for when running on the wayland platform #3614

Bugs fixed:
. SIGSEGV on input device disconnection #3601
. Frame fails to enforce fullscreen on wpe-webkit-mir-kiosk #3600
. Chromium unmaximises when focus is lost #3592
. Apps constantly resizing #3573
. Our fork/exec spawning is unsafe #3494
. Revert "Fix random leak" #3609
. Smoke tests are failing #3610
Saviq added a commit that referenced this pull request Sep 27, 2024
Enhancements:
. Allow to specify an app id for when running on the wayland platform #3614

Bugs fixed:
. SIGSEGV on input device disconnection #3601
. Frame fails to enforce fullscreen on wpe-webkit-mir-kiosk #3600
. Chromium unmaximises when focus is lost #3592
. Apps constantly resizing #3573
. Our fork/exec spawning is unsafe #3494
. Revert "Fix random leak" #3609
. Smoke tests are failing #3610
@Saviq Saviq mentioned this pull request Sep 27, 2024
github-merge-queue bot pushed a commit that referenced this pull request Sep 30, 2024
>  Release 2.18.2
>
>  Enhancements:
>  - #3614
>  - #3617
>
>  Bugs fixed:
>   - #3601
>   - #3600
>   - #3592
>   - #3573
>   - #3494
>   - #3609
>   - #3610
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.

3 participants