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 733684 - Nautilus tabs are bigger than any other application
Nautilus tabs are bigger than any other application
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: Tabs
3.13.x
Other Linux
: Normal normal
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2014-07-24 15:19 UTC by alex diavatis
Modified: 2014-09-04 22:37 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
nautilus tabs (703.07 KB, image/png)
2014-07-24 15:19 UTC, alex diavatis
  Details
Fix Adwaita.css styles for tabs (561 bytes, patch)
2014-08-18 15:40 UTC, alex diavatis
committed Details | Review
Screen after patch (417.51 KB, image/png)
2014-08-18 15:42 UTC, alex diavatis
  Details
without hover (429.95 KB, image/png)
2014-08-18 21:28 UTC, alex diavatis
  Details
nautilus 3.13.91 tabs (69.43 KB, image/png)
2014-09-04 18:30 UTC, Eddy Castillo
  Details

Description alex diavatis 2014-07-24 15:19:08 UTC
Created attachment 281608 [details]
nautilus tabs

Bug is visible in screenshot
Comment 1 António Fernandes 2014-07-24 20:39:53 UTC
Also, the blue line is missing from the selected tab.
Comment 2 Lapo Calamandrei 2014-08-08 11:27:41 UTC
nautilus needs apps specific fixes for that, which are not in the theme anymore, so it needs to ship the css for that.
That specific issue can be fixed with this css:

NautilusNotebook.notebook tab { border-width: 0px 1px 3px; }
NautilusNotebook.notebook tab .button { padding: 0px; }
Comment 3 alex diavatis 2014-08-18 15:40:55 UTC
Created attachment 283788 [details] [review]
Fix Adwaita.css styles for tabs
Comment 4 alex diavatis 2014-08-18 15:42:26 UTC
Created attachment 283789 [details]
Screen after patch
Comment 5 Cosimo Cecchi 2014-08-18 21:20:29 UTC
I don't think that looks entirely right either, the tabs should not that space between them. Lapo, can you take a look?
Comment 6 alex diavatis 2014-08-18 21:23:52 UTC
(In reply to comment #5)
> I don't think that looks entirely right either, the tabs should not that space
> between them. Lapo, can you take a look?

It is a standard GTK theming space, in all apps.
Comment 7 alex diavatis 2014-08-18 21:27:39 UTC
(In reply to comment #6)
> (In reply to comment #5)
> > I don't think that looks entirely right either, the tabs should not that space
> > between them. Lapo, can you take a look?
> 
> It is a standard GTK theming space, in all apps.

It happens because I have in active Tab in hover
Comment 8 alex diavatis 2014-08-18 21:28:50 UTC
Created attachment 283830 [details]
without hover
Comment 9 Cosimo Cecchi 2014-08-18 22:06:30 UTC
Review of attachment 283788 [details] [review]:

Looks good
Comment 10 Cosimo Cecchi 2014-08-18 22:07:00 UTC
(In reply to comment #7)
> It happens because I have in active Tab in hover

Oh that explains it then... Patch looks good to me.
Comment 11 Lapo Calamandrei 2014-08-21 08:53:24 UTC
Cosimo the gap between tabs is intended. Fix looks good to me.
Comment 12 Cosimo Cecchi 2014-08-28 00:33:20 UTC
Pushed to master now.
Comment 13 Eddy Castillo 2014-09-04 18:30:12 UTC
Created attachment 285404 [details]
nautilus 3.13.91 tabs

This issue is still on 3.13.91. I downloaded 3.13.91 tarball and Adwaita.css doesn't have this fix. Was it removed in another commit?
Comment 14 alex diavatis 2014-09-04 19:15:11 UTC
(In reply to comment #13)
> Created an attachment (id=285404) [details]
> nautilus 3.13.91 tabs
> 
> This issue is still on 3.13.91. I downloaded 3.13.91 tarball and Adwaita.css
> doesn't have this fix. Was it removed in another commit?

I don't know what you're missing, you need both latest GTK+ and Nautilus. It seems to work fine from upstream
Comment 15 Eddy Castillo 2014-09-04 22:37:44 UTC
Yeah, it was a previous version of GTK+. It doesn't happen with the latest version. I'm sorry for the noise.