Problem/Motivation

The preview button in Claro theme is not correctly displayed, the link text and icon are partially rendered on top of the image preview as shown in the following screenshot:

Focal point preview button screenshot in Claro theme

Steps to reproduce

Just install the module and enable the image preview.

Proposed resolution

Basic review of the preview button markup by adding the button classes and change some custom CSS rules.

Remaining tasks

Review, commit.

User interface changes

None.

API changes

N/A

Data model changes

N/A

Comments

manuel.adan created an issue. See original summary.

manuel.adan’s picture

After the proposed patch:

Focal point preview button screenshot in Claro theme after the proposed patch

Before in the Seven theme:

Focal point preview button screenshot in Seven theme

After the proposed patch in the Seven theme:

Focal point preview button screenshot in Seven theme

bleen’s picture

Status: Needs review » Needs work
StatusFileSize
new187.15 KB

I like this, but ...

  • When its used with a standard image field (not a media field) it makes the button a bit large as compared to the thumbnail
  • Similarly it can get confused with the "preview" button at the bottom of the node add form

Things to think about:

  • ways to differentiate the button from the other preview button
  • I'm open to changing the text to "Image Preview" or something, but if its too long thats not good either
johan den hollander’s picture

StatusFileSize
new85.42 KB
new95.97 KB
new100.96 KB
new317 bytes

I came here because the preview link is not placed nicely in Claro (and Gin).
Therefore I made a simple patch, moving the link down a bit, just far enough to look good on Claro and Gin, and still looking ok for seven.

The patch adds just these two lines to .focal-point-preview-link:

position: relative;
top:13px;

Seven after patching
Seven patched

Claro after patching
Claro patched

Gin after patching
Gin patched

berdir’s picture

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

Looks like this was fixed in #3265251: Preview link overlaps with image on Claro theme in a different way, can you confirm?

martijn de wit’s picture

berdir’s picture

Status: Postponed (maintainer needs more info) » Closed (duplicate)