After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 748306 - Don't mix raised buttons with non-raised ones
Don't mix raised buttons with non-raised ones
Status: RESOLVED FIXED
Product: totem
Classification: Core
Component: Movie player
3.14.x
Other Linux
: Normal normal
: ---
Assigned To: General Totem maintainer(s)
General Totem maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2015-04-22 15:22 UTC by Allan Day
Modified: 2015-08-31 15:01 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
backend: Don't flatten OSD buttons (1.34 KB, patch)
2015-08-28 10:08 UTC, Bastien Nocera
none Details | Review
osd toolbar (57.55 KB, image/png)
2015-08-28 10:58 UTC, Bastien Nocera
  Details

Description Allan Day 2015-04-22 15:22:16 UTC
The playback controls bar includes a mixture of buttons styles - some buttons are raised (ie. have relief), while others don't. This is a bit inconsistent.

Buttons generally have relief in GNOME 3 - it would be best to use relief on all of them in the play controls bar.
Comment 1 Bastien Nocera 2015-08-28 10:08:19 UTC
Created attachment 310166 [details] [review]
backend: Don't flatten OSD buttons
Comment 2 Bastien Nocera 2015-08-28 10:11:29 UTC
Two problems though.

First, Adwaita adds the "flat" CSS style to the volume button at the end of the OSD, probably because it's the last item in the toolbar (?).

Second, even if you manage to fix that, because the buttons aren't linked, it looks bad. We need to add some padding or something.
Comment 3 Allan Day 2015-08-28 10:52:22 UTC
(In reply to Bastien Nocera from comment #2)
> Two problems though.
> 
> First, Adwaita adds the "flat" CSS style to the volume button at the end of
> the OSD, probably because it's the last item in the toolbar (?).
>
> Second, even if you manage to fix that, because the buttons aren't linked,
> it looks bad. We need to add some padding or something.

Got a screenshot of this?
Comment 4 Bastien Nocera 2015-08-28 10:58:55 UTC
Created attachment 310176 [details]
osd toolbar
Comment 5 Bastien Nocera 2015-08-28 11:11:21 UTC
Fixed in a different way in commit e562f02
Comment 6 Lapo Calamandrei 2015-08-29 11:40:23 UTC
Just for future reference, the volume button default look is flat, so this is not actually a bug.
Comment 7 Bastien Nocera 2015-08-29 17:47:14 UTC
(In reply to Lapo Calamandrei from comment #6)
> Just for future reference, the volume button default look is flat, so this
> is not actually a bug.

What isn't actually a bug?
Comment 8 Bastien Nocera 2015-08-29 17:47:40 UTC
Did you mean to comment on bug 754225 instead?
Comment 9 Lapo Calamandrei 2015-08-29 22:20:50 UTC
gaaah, exactly Bastien, sorry for the noise. Mental note, don't comment bug using the phone...
Comment 10 Bastien Nocera 2015-08-31 14:17:13 UTC
So, should the volume button have a flat look or not?

It currently looks bad right next to the "..." button.
Comment 11 Bastien Nocera 2015-08-31 15:01:24 UTC
commit 5208baf642701757a0f6de5030183734b93fc3f0
Author: Bastien Nocera <hadess@hadess.net>
Date:   Mon Aug 31 16:57:44 2015 +0200

    backend: Re-flatten the volume button
    
    It's supposed to be flat.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=748306