search

GavickPro / Portfolio-Free-Ghost-Theme

Simple portfolio theme for Ghost
MIT License
126 stars 39 forks source link

Clickable Image #8

Closed s0eren closed 9 years ago

s0eren commented 10 years ago

Hej, I was wondering if there is a possibility that on the default-page the whole image (and not only the text) is clickable to read the whole article.

thanks!

dziudek commented 10 years ago

Hello, but what is the sense of it, if the image is always covered by the overlay with the linked text?

s0eren commented 10 years ago

It's just that you have to explicity hover over some text for the image to be 'clickable'. I just thought it would be nice if the whole text overlay would be linked to the article and not just some lines with a whole lot of 'unused' whitespace.

unwitting commented 10 years ago

+1. The point is that if I hover over, say, the very top-right corner of a post, where the image was until the overlay appeared, I'm not hovering over a clickable link. I feel like I should be, and have certainly tried a click or two there now.

With that in mind, I'd recommend wrapping the whole div.post__wrapper_helper in the link, not just some of the text elements within the overlay?

dziudek commented 10 years ago

Ok, I will consider a solution for it in the next update of the theme.

dziudek commented 9 years ago

Added: https://github.com/GavickPro/Portfolio-Free-Ghost-Theme/commit/d83510098950dce9d229e31082b0d5c9aab79aa6