evince should display CSD headerbars for budgie-desktop
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
evince (Ubuntu) |
Fix Released
|
Medium
|
Unassigned |
Bug Description
budgie-desktop should display GNOME based CSD headerbars.
Due to the patching, it displays non GNOME based menus.
Attached is a patch to resolve this. It has been tested on multiple desktop environments;
It correctly displays a traditional menu bar for Unity.
For both GNOME-Shell and budgie-desktop CSD is displayed.
Note - I have taken the clean and consistent approach as the patching for gedit.
The extant traditional and unity patches are confused - DESKTOP_SESSION comparison is hard-coded for GNOME - budgie-desktop uses the DESKTOP_SESSION of BUDGIE:GNOME. The traditional menu bar patch is inconsistent and uses a completely different algorithm that doesnt work with budgie-desktop.
I have simplified the logic to just apply CSD for GNOME based desktops as per gedit patching.
Related branches
- Jeremy Bícha: Approve
-
Diff: 278 lines (+95/-63)2 files modifieddebian/patches/traditional_menu_bar.patch (+34/-51)
debian/patches/unity_normal_titlebar.patch (+61/-12)
Changed in evince (Ubuntu): | |
importance: | Undecided → Medium |
jbicha requested a merge proposal against lp:~ubuntu-desktop/evince/ubuntu so have created a branch as well