evince should display CSD headerbars for budgie-desktop

Bug #1670054 reported by fossfreedom
8
This bug affects 1 person
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.

Tags: zesty

Related branches

Revision history for this message
fossfreedom (fossfreedom) wrote :
Changed in evince (Ubuntu):
status: New → In Progress
assignee: nobody → fossfreedom (fossfreedom)
Mathew Hodson (mhodson)
Changed in evince (Ubuntu):
importance: Undecided → Medium
Revision history for this message
fossfreedom (fossfreedom) wrote :

jbicha requested a merge proposal against lp:~ubuntu-desktop/evince/ubuntu so have created a branch as well

Revision history for this message
Jeremy Bícha (jbicha) wrote :

Oops, I didn't see this bug before I uploaded.

https://launchpad.net/ubuntu/+source/evince/3.22.1-3ubuntu1

You are welcome to add a debian/changelog entry (but leave release as UNRELEASED) when proposing merges for ubuntu-desktop packaging.

Changed in evince (Ubuntu):
assignee: fossfreedom (fossfreedom) → nobody
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.