incubator-bloodhound-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gary Martin <gary.mar...@wandisco.com>
Subject Re: Ticket #279 - Images rendered in activity feed
Date Wed, 19 Dec 2012 13:15:31 GMT
Great to hear from you Tom!

I think for a quick solution I could live with not displaying in the 
short term. Alternatively, swapping the image for a placeholder image 
might work.

I could see the visibility of images as being something configurable 
though. For tall images, perhaps we could just scale them back to some 
maximum height.

Cheers,
Gary

On 19/12/12 12:58, Peter Koželj wrote:
> I am in favor of keeping images. Normally they should not be to heigh when
> resized to fit activity feed width.
> In the long run I would like if users have an option on activity widget or
> in widget activity setting to show/hide images.
>
> And...
> ... welcome to Bloodhound!
>
> Peter
>
> On 19 December 2012 13:05, Tom Kitchin <tkitchin@gmail.com> wrote:
>
>> Hi,
>>
>> I've been reading the dev mailing list for a while, and I thought it might
>> be a time to try and get involved.  I took a look at ticket #279 (
>> https://issues.apache.org/bloodhound/ticket/279), in which images in
>> ticket descriptions turn up in the activity feed as well.
>>
>> The first solution (not rendering them) is simple enough as far as I can
>> see - I've attached a very simple patch which just sets them display:none
>> in CSS.
>>
>> I've been looking into the second solution (resizing the images to fit
>> instead of hiding them) as well.  As far as I can tell (prompted by Gary)
>> the trick would be to do something with the Image() macro in the specific
>> context of the Activity feed to apply the 'span4' class to the img tag.  I
>> haven't quite worked out how to do that yet, but thought I'd send a message
>> here before I persisted.
>>
>> Which approach is preferred?  As far as I can see, the question comes down
>> to whether the activity feed could end up misinforming people due to
>> missing images, and whether it would look jarring if a referenced image
>> wasn't there.  On the other hand, even resized images might still be very
>> long, taking up a lot of space in the feed.
>>
>> Incidentally, nice to meet you all.  I hope I can be of some help with the
>> project.
>>
>> Tom
>>


Mime
View raw message