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 733374 - Inline HeaderBar looks bad in theme with rounded corners
Inline HeaderBar looks bad in theme with rounded corners
Status: RESOLVED FIXED
Product: geary
Classification: Other
Component: composer
master
Other Linux
: Normal minor
: ---
Assigned To: Geary Maintainers
Geary Maintainers
review
Depends on:
Blocks:
 
 
Reported: 2014-07-18 17:10 UTC by Robert Schroll
Modified: 2014-07-23 02:25 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Screenshot showing appearance in (a varient of) the Ambiance theme (8.10 KB, image/png)
2014-07-18 17:10 UTC, Robert Schroll
  Details
Force inline ComposerHeaderbar to have square corners (1.55 KB, patch)
2014-07-19 23:52 UTC, Robert Schroll
none Details | Review
Force inline ComposerHeaderbar to have square corners (1.64 KB, patch)
2014-07-20 15:14 UTC, Robert Schroll
committed Details | Review

Description Robert Schroll 2014-07-18 17:10:41 UTC
Created attachment 281137 [details]
Screenshot showing appearance in (a varient of) the Ambiance theme

Some themes give HeaderBars rounded top corners.  This doesn't look good when we use them for inline composers, since we're drawing a rectangular border and shadow behind them.  (See attached screenshot.)

This may be a bug in the theme.  I note that in Adiawata, HeaderBars on Windows are rounded, but those in other widgets are not.

There are several lines of attach we could use:
- Modify the theming so that inline HeaderBars have sharp corners.
- Detect the border radius in GTK and match it with the underlying HTML.
- Don't draw the border and shadow in HTML, and do it in GTK instead.
Comment 1 Robert Schroll 2014-07-19 23:52:22 UTC
Created attachment 281212 [details] [review]
Force inline ComposerHeaderbar to have square corners

This takes the first approach, modifying the GTK CSS to make HeaderBars 
that appear in Gtk.Alignments have square corners.  It does this through 
a global style provider.  I think we could also do this via a local 
style provider acting only on HeaderBars; I don't know if that'd be 
better or not.  We may also want to see about making sure this is active 
when we're running under ENABLE_UNITY; I noticed that also looked bad 
with rounded corners under Ambiance.
Comment 2 Robert Schroll 2014-07-20 15:14:02 UTC
Created attachment 281239 [details] [review]
Force inline ComposerHeaderbar to have square corners

This tweaks the CSS a bit to also catch the main window HeaderBar when 
we're running with ENABLE_UNITY.  Rounded corners look even worse then.
Comment 3 Jim Nelson 2014-07-22 19:46:30 UTC
Review of attachment 281239 [details] [review]:

I'm fine with this approach.  Commit!
Comment 4 Robert Schroll 2014-07-23 02:25:12 UTC
Attachment 281239 [details] pushed as b6bcbb9 - Force inline ComposerHeaderbar to have square corners