When a node is unpublished in Bartik, the pink background extends outwards and overlaps part of the tab navigation, which doesn't look right to me.

Not sure if this is intentional, or just a minor oversight, so please feel free to close as won't fix if this was a design decision.

Current state:
Current state

Without the margin: -20px -15px 0; CSS it looks a bit better in my opinion:
Without negative margin

CommentFileSizeAuthor
#4 issue-2346026-4-after.png41.66 KBckrina
#4 issue-2346026-4-before.png45.19 KBckrina
#1 issue-2346026-1.patch364 bytessqndr
Screen Shot 2014-09-27 at 18.44.25.png37.42 KBgeodaniel
Screen Shot 2014-09-27 at 18.44.48.png36.35 KBgeodaniel

Comments

sqndr’s picture

sqndr commented
StatusFileSize
new issue-2346026-1.patch364 bytes

Here's a patch removing the margin:

Before: http://cl.ly/image/3Z0F2S1E3k1t
After: http://cl.ly/image/132w160X2G2z

geodaniel’s picture

geodaniel commented
Status: Active » Needs review

Looks good to me - thanks for the quick patch!

sqndr’s picture

sqndr commented
Issue tags: +Amsterdam2014

Tagging with Amsterdam2014.

ckrina’s picture

ckrina
she/her
Primary language Catalan
Location Barcelona
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new issue-2346026-4-before.png45.19 KB
new issue-2346026-4-after.png41.66 KB

I've just tested it and the #1 patch solves the problem.

creatile’s picture

creatile commented

I agree, patch works well

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Good catch.

Committed and pushed to 8.x. Thanks!

  • webchick committed a741c86 on 8.0.x 
    Issue #2346027 by sqndr, ckrina, geodaniel: Fixed Unpublished content...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.