Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lsolesen’s picture

Component: Code » Widgets
Status: Active » Needs review
FileSize
1.13 KB

See result on vies.dk :)

mrfelton’s picture

Status: Needs review » Postponed (maintainer needs more info)

@lsolesen can you explain what the problem is here and why this patch is needed please?

lsolesen’s picture

Because the width looks better after applying the patch :)

Before:

After:

lsolesen’s picture

Status: Postponed (maintainer needs more info) » Needs review
mrfelton’s picture

I can't replicate the issue with the default responsive garland theme. Could this be caused by the theme you are using?

lsolesen’s picture

@mrfelton --> It could be caused by Radix (which is my basetheme then). I will investigate further.

mrfelton’s picture

Status: Needs review » Postponed (maintainer needs more info)
dsnopek’s picture

Status: Postponed (maintainer needs more info) » Fixed

I'm able to reproduce this problem when using Panopoly with radix_layouts (which we're going to start using per #2191069: Have Panopoly Theme depend on Radix Layouts (rather than providing it's own)) and responsive_bartik - so, not just with Radix. The attached patch improves the situation! Thanks, @lsolesen! Committed.

  • dsnopek committed b4edd49 on 7.x-1.x
    Update Panopoly Widgets for #2163985 by lsolesen: Added Too narrow...
dsnopek’s picture

Status: Postponed (maintainer needs more info) » Fixed

Hrm, setting to Fixed again!

Status: Fixed » Closed (fixed)

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