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

Re-add X-GNOME-SingleWindow #23875

Merged

Conversation

TheEvilSkeleton
Copy link

X-GNOME-SingleWindow is best to be used for backward compatibility. Since SingleMainWindow was recently added, many Desktop Environments and Window Managers may not have it implemented yet.

@CLAassistant
Copy link

CLAassistant commented Jan 9, 2022

CLA assistant check
All committers have signed the CLA.

@ilya-fedin
Copy link
Contributor

Can you move SingleMainWindow before the X-GNOME keys?

@TheEvilSkeleton
Copy link
Author

Done.

@ilya-fedin
Copy link
Contributor

No, before the both keys

@TheEvilSkeleton
Copy link
Author

Oh, sorry about that 😅

`X-GNOME-SingleWindow` is best to be used for backward compatibility. Since `SingleMainWindow` was recently added, many Desktop Environments and Window Managers may not have it implemented yet.

This MR also moves `SingleMainWindow` before the `X-GNOME-*` keys, as requested in this comment: telegramdesktop#23875 (comment)
@TheEvilSkeleton
Copy link
Author

Like this? 39bde7f

@ilya-fedin
Copy link
Contributor

Yeah

@xvitaly
Copy link
Contributor

xvitaly commented Jan 10, 2022

$ desktop-file-validate telegramdesktop.desktop
telegramdesktop.desktop: error: file contains key "SingleMainWindow" in group "Desktop Entry", but keys extending the format should start with "X-"

SingleMainWindow should be deleted.

@refi64
Copy link

refi64 commented Jan 10, 2022

@xvitaly
Copy link
Contributor

xvitaly commented Jan 10, 2022

desktop-file-utils does not have support for the 1.5 standard yet.

Running this check is mandatory for the most of GNU/Linux distributions and Flatpak. Maintainers will need to manually remove this row.

Let's discuss that in #23879.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants