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

Linux: Window doesn't close/quit on first click #12342

Open
core-ai-bot opened this issue Aug 31, 2021 · 20 comments
Open

Linux: Window doesn't close/quit on first click #12342

core-ai-bot opened this issue Aug 31, 2021 · 20 comments

Comments

@core-ai-bot
Copy link
Member

Issue by jasonsanjose
Tuesday Jul 30, 2013 at 23:29 GMT
Originally opened as adobe/brackets#4611


On Ubuntu, when attempting to close the Brackets window via the X button, it takes 2 attempts before the window actually closes.

@core-ai-bot
Copy link
Member Author

Comment by timburgess
Wednesday Jul 31, 2013 at 01:11 GMT


I have done some investigation into this. It appears that earlier versions of the linux CEF3 didn't emit a GTK delete event - this has now been patched:
https://code.google.com/p/chromiumembedded/issues/attachmentText?id=780&aid=7800000000&name=gtk_delete_event.diff&token=-r_yEWKd9SaWYQGP9URC7I91jxI%3A1374642422808

The present code has a workaround which now that we are using patched CEF3 versions results in multiple clicks required.

@core-ai-bot
Copy link
Member Author

Comment by bradberger
Thursday Oct 17, 2013 at 09:07 GMT


Also having a similar problem with newly installed Sprint 32 (sprint 32 experimental build 0.32.0-9791 (On branch master,master e49a849fb) installed via deb package on Xubuntu 13.04. In fact, it's not closing at all. As a result, need to kill both brackets and brackets-node processes.

This didn't occur with Sprint 31 on the same system, so perhaps something's changed recently. Thanks!

@core-ai-bot
Copy link
Member Author

Comment by redmunds
Thursday Dec 26, 2013 at 16:50 GMT


@dixonge The core team is short on native Linux skills. Would you like to try to fix this one?

@core-ai-bot
Copy link
Member Author

Comment by fakam
Tuesday Feb 11, 2014 at 19:56 GMT


Same thing on Ubuntu 13.10 / Mint 16 64bit

@core-ai-bot
Copy link
Member Author

Comment by jdittrich
Friday Feb 28, 2014 at 09:26 GMT


same here 12.04, Brackets 36

Are there any additional infos on that to ease the assessment of the problem for people who want to try to fix it? If I understand@timburgess correctly, the problem is caused by code in Brackets which was a workaround for a now-fixed bug in CEF?

@core-ai-bot
Copy link
Member Author

Comment by peterflynn
Thursday Mar 06, 2014 at 20:17 GMT


Looks like someone should try uncommenting the disabled code in the Linux impl of CloseWindow(): https://github.com/adobe/brackets-shell/blob/master/appshell/appshell_extensions_gtk.cpp#L479

UPDATE: the "disabled code" referred to above now seems to be on a different line:
https://github.com/adobe/brackets-shell/blob/master/appshell/appshell_extensions_gtk.cpp#L531

Anyone want to give that a try and send a PR if it works?

@core-ai-bot
Copy link
Member Author

Comment by peterflynn
Thursday Jun 19, 2014 at 09:38 GMT


Note: Dupe #5714 (or #7794) has some steps where this could cause data loss. Marking 'needs review' to consider whether the core team should take cycles to work on this.

@core-ai-bot
Copy link
Member Author

Comment by dangoor
Monday Jun 23, 2014 at 20:36 GMT


Assigned to@redmunds to follow up with community member who might be interested in this.

@core-ai-bot
Copy link
Member Author

Comment by sryze
Wednesday Oct 29, 2014 at 06:50 GMT


I have this problem too, Brackets 0.44 on Ubuntu 14.04

@core-ai-bot
Copy link
Member Author

Comment by peterflynn
Sunday Jan 18, 2015 at 12:18 GMT


@redmunds Is someone working on this? It looks like you added 'fix in progress' recently but also cleared the assignee...

@core-ai-bot
Copy link
Member Author

Comment by redmunds
Sunday Jan 18, 2015 at 18:18 GMT


@peterflynn The pull request is adobe/brackets-shell#458. This was assigned to me, but I have not been involved with it, so I removed myself.

@core-ai-bot
Copy link
Member Author

Comment by MrSkwiggs
Sunday Apr 19, 2015 at 10:19 GMT


This bug still exists in Brackets Release 1.2 experimental build 1.2.0-15697 (release 8f82e2a97)
and Release 1.3 experimental build 1.3.0-012 (release 2fbd9d392) .
Also, when reopening Brackets, it reverts to a smaller window size even if it was fullscreen before exiting...

@core-ai-bot
Copy link
Member Author

Comment by rthouvenin
Tuesday Feb 09, 2016 at 09:06 GMT


Anything blocking the review / merge of the PR? The fix seems to work according to@jasonsanjose

@core-ai-bot
Copy link
Member Author

Comment by robinpecha
Saturday Feb 13, 2016 at 22:55 GMT


Ubuntu 14.04 - Brackets 1.6 bug still exists.
Does someone have clear tip how to patch it right now?

@core-ai-bot
Copy link
Member Author

Comment by jextrevor
Wednesday Nov 02, 2016 at 11:29 GMT


Still having this issue, Ubuntu 16.04 - Brackets 1.7. I concur with@Mr-Skwiggs - the window does revert to a smaller size after reopening, even if it was fullscreen.

@core-ai-bot
Copy link
Member Author

Comment by zaggino
Wednesday Nov 02, 2016 at 12:03 GMT


@jextrevor try brackets-electron, https://github.com/zaggino/brackets-electron/releases current version matches latest brackets and works much better on linux (minus live preview functionality)

@core-ai-bot
Copy link
Member Author

Comment by axedre
Wednesday Nov 02, 2016 at 13:10 GMT


Looks neat, will wait for some good soul to include it in AUR since I use arch linux...

@core-ai-bot
Copy link
Member Author

Comment by zaggino
Wednesday Nov 02, 2016 at 13:45 GMT


@axedre you should be able to install AppImage on arch linux as far as I know. More info on http://appimage.org/

@core-ai-bot
Copy link
Member Author

Comment by cybbro
Tuesday Aug 01, 2017 at 08:28 GMT


Still having this issue on Brackets 1.10 on Ubuntu GNOME 17.04 64bit, I always have to press close two times to exit the program.

@core-ai-bot
Copy link
Member Author

Comment by saurabh95
Wednesday Aug 23, 2017 at 10:07 GMT


Fixed By: adobe/brackets-shell#619

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

No branches or pull requests

1 participant