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: handle gjs in 3.36 #1131

Merged
merged 1 commit into from
Aug 9, 2021
Merged

fix: handle gjs in 3.36 #1131

merged 1 commit into from
Aug 9, 2021

Conversation

jmmaranan
Copy link
Contributor

@jmmaranan jmmaranan commented Aug 6, 2021

Fixes #1056

@jacobgkau jacobgkau mentioned this pull request Aug 6, 2021
@jacobgkau jacobgkau requested a review from a team August 6, 2021 21:15
@jmmaranan jmmaranan closed this Aug 9, 2021
@jmmaranan jmmaranan deleted the fix-gjs-stray-iconw branch August 9, 2021 14:53
@jmmaranan jmmaranan restored the fix-gjs-stray-iconw branch August 9, 2021 14:53
@jmmaranan jmmaranan reopened this Aug 9, 2021
@jmmaranan
Copy link
Contributor Author

I think Github needs a confirm dialog when cleaning up branches. I re-opened the PR

Copy link
Member

@jacobgkau jacobgkau left a comment

Choose a reason for hiding this comment

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

Confirmed this fixes the stray gjs icon in Activities, recreated by installing the WindowOverlay Icons extension on a 20.04 LTS system.

Regression testing passed:
shell-pr1131-hirsute.txt
shell-pr1131-focal.txt

@jacobgkau jacobgkau requested a review from a team August 9, 2021 16:23
@jacobgkau
Copy link
Member

Sorry, I forgot to mention that this does not fix #1094, so it should not be closed by this PR.

@jmmaranan
Copy link
Contributor Author

I removed the ticket 1094. I thought the emulator was its own window (not transient with) from the IDE.

@mmstick mmstick merged commit 3eda9bb into pop-os:master Aug 9, 2021
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.

gjs file running
3 participants