Ubuntu theme

Merge lp://staging/~c-lobrano/ubuntu-themes/fix-1704745 into lp://staging/ubuntu-themes

  1. fix-1704745
  2. Merge into trunk
Proposed by Carlo Lobrano
Status: Rejected
Rejected by: Marco Trevisan (Treviño)
Proposed branch: lp://staging/~c-lobrano/ubuntu-themes/fix-1704745
Merge into: lp://staging/ubuntu-themes
Diff against target: 61 lines (+38/-1)
2 files modified
Ambiance/gtk-3.20/apps/libreoffice.css (+19/-1)
Radiance/gtk-3.20/apps/libreoffice.css (+19/-0)
To merge this branch: bzr merge lp://staging/~c-lobrano/ubuntu-themes/fix-1704745
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Needs Fixing
Review via email: mp+327584@code.staging.launchpad.net

Commit message

Fix Libreoffice close button style

In both Ambiance and Radiance, the Libreoffice's close button
on the tob right corner (below the windows's close button) has
a different color respect the menu bar in which is placed.

This commit fixes close button color and background in normal,
unfocused and hover state.

To post a comment you must log in.
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

I'm sorry we can't apply this as it's too generic and it breaks (for example) gitg commit message.

I don't think we can't do much here until libreoffice doesn't add a (parent, at least) widget with its class name

review: Needs Fixing
Revision history for this message
Carlo Lobrano (c-lobrano) wrote :

Oh, I see, that's a pity

Revision history for this message
Adolfo Jayme Barrientos (fitojb) wrote :

Is the LibreOffice issue reported upstream? If not, please do :) See https://bugs.documentfoundation.org/show_bug.cgi?id=GTK3

Revision history for this message
Carlo Lobrano (c-lobrano) wrote :

Hi Adolfo,

it doesn't seem necessary anymore, it is not reproducible anymore in LibreOffice 5.4

Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

I think the main issue here is still the fact that the selector is too general and also other apps might be affected by change.

Revision history for this message
Carlo Lobrano (c-lobrano) wrote :

I see, thanks for the new review anyway. If I'll reproduce the bug on the new LibreOffice 5.4, I'll open a bug upstream, then.

Unmerged revisions

556. By Carlo Lobrano

Fix LibreOffice close button style

improved previous fix, using a rule more specific that (should)
avoid to break other applications (e.g. gitg).

This commit makes the background and foreground color of the
LibreOffice's close button on the top right corner matching
with the adiacent menubar.

Fixes LP:1704745

555. By Carlo Lobrano

Update from mainline

554. By Carlo Lobrano

Fix Libreoffice close button style

In both Ambiance and Radiance, the Libreoffice's close button
on the tob right corner (below the windows's close button) has
a different color respect the menu bar in which is placed.

This commit fixes close button color and background in normal,
unfocused and hover state.

Fixes LP:1704745

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches