Merge lp:~vikoadi/scratch/fullscreen-toolbar into lp:~elementary-apps/scratch/scratch
Proposed by
Viko Adi Rahmawan
Status: | Merged |
---|---|
Approved by: | Cody Garver |
Approved revision: | 1321 |
Merged at revision: | 1320 |
Proposed branch: | lp:~vikoadi/scratch/fullscreen-toolbar |
Merge into: | lp:~elementary-apps/scratch/scratch |
Diff against target: |
153 lines (+20/-14) 8 files modified
plugins/folder-manager/FolderManagerPlugin.vala (+2/-1) plugins/pastebin/pastebin.vala (+2/-1) plugins/pastebin/pastebin_dialog.vala (+3/-1) src/Dialogs/PreferencesDialog.vala (+4/-2) src/MainWindow.vala (+5/-4) src/Services/Document.vala (+1/-1) src/Utils.vala (+2/-2) src/scratch-ui.xml (+1/-2) |
To merge this branch: | bzr merge lp:~vikoadi/scratch/fullscreen-toolbar |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Danielle Foré | Needs Fixing | ||
Robert Roth (community) | code review | Approve | |
Review via email: mp+222239@code.launchpad.net |
Commit message
Remove fullscreen from appmenu but still leave F11 Fullscreen key
Window state saving fix
Description of the change
remove fullscreen from appmenu
also fix scratch to save Gdk.WindowState properly
To post a comment you must log in.
Thanks for the proposal, the fullscreen header is mostly working.
However, when opening the preferences while in fullscreen, the preferences dialog is shown, and the main window is hidden, and it does not reappear when the preferences dialog is closed, have to switch back to the application with Alt+Tab.
Could you please fix this?